Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fluid disk shown as thousand blocks in disk drive #370

Open
WhiteBoi1211 opened this issue Apr 27, 2023 · 7 comments
Open

Fluid disk shown as thousand blocks in disk drive #370

WhiteBoi1211 opened this issue Apr 27, 2023 · 7 comments
Labels
enhancement New feature or request

Comments

@WhiteBoi1211
Copy link

Describe the bug

The fluid disk shows as blocks in the disk drive. Ex: picture, the 256B disk shows as 256.000 blocks/items.

2023-04-27_16 16 44
2023-04-27_16 16 47
2023-04-27_16 16 50

(Minor issue yes, just wanted to point it out incase it wasn't known! :D)

How can we reproduce this bug or crash?

Simply by putting a fluid disk into a disk drive

What Minecraft version is this happening on?

Minecraft 1.19.3

What Forge or Fabric version is this happening on?

44.1.0

What Refined Storage version is this happening on?

"2.0.0-milestone.2.8"

Relevant log output

No response

@WhiteBoi1211 WhiteBoi1211 added the bug Something isn't working label Apr 27, 2023
@jonatjano
Copy link

since the disk drive allow both types of disks (like it did in RS), having 2 progress bars (eventually showing only if disk type is present) could make the interface better

@raoulvdberge
Copy link
Contributor

I thought about that but if you get more types, you'll get more progress bars (think about future Mekanism gas support for example, you could end up with 3 progress bars)

@WhiteBoi1211
Copy link
Author

3 bars wouldn't be a problem; move the disk slots to the left and you'll have space. You shouldn't need any more than items, liquids, and gasses

@jonatjano
Copy link

There could be compatibility with other mods, I have in mind botania mana (mana from many mods tbh), maybe someone would want to put a battery addon

I don't know how much it is feasible using minecraft GUI api, but there are algorithms which takes the number of elements you want to show and return an appropriate number of element per line to have a nice grid

For this case I'd personally use something along the line of
countPerLine = ceil(sqrt(elementCount) + epsilon) + 1
which gives a nice rectangle

elementCount countPerLine note
1 2
2 3
3 3
4 4
5 4 second line appear from here
6 4
7 4
8 4
9 5
10 5
11 5 third line appear from here

Having 10 different kind of resources would already be madness imo, but with this technique it should be acceptably rendered

@raoulvdberge
Copy link
Contributor

@jonatjano what is a "line"? How would it look like in the UI?

@jonatjano
Copy link

here's what I have in mind,
when having up to 4 different type of resources, it only adds progress bar
upTo4

starting from 5 type, the progress bar get squished by half their height and a new line of progress bar is added
from5

the color of the progress bar helps recognising them easily


While it's true that the UI becomes a bit overpopulated when you have the second line, it is still more than readable

@raoulvdberge raoulvdberge added enhancement New feature or request and removed bug Something isn't working labels Apr 1, 2024
@adroslice
Copy link

Another option might be to paginate progress bars or even have a single bar and button that cycles through all the possible fill types. Obviously, this means not being able to identify a nearly full disk at a glance, but this is still indicated by the LED color on the model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo
Development

No branches or pull requests

4 participants