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

Rearrange Super and Quantum Tank GUI a little bit #2084

Merged
merged 1 commit into from Jun 15, 2023

Conversation

wlhlm
Copy link
Member

@wlhlm wlhlm commented Jun 15, 2023

After some complaining in Discord, this brings fluid contents and lock slot on the same line and moves the voiding buttons below the lock slot.

2023-06-15_17 26 23

@github-actions
Copy link
Contributor

Warning: 2 uncommitted changes
#2085

@Dream-Master
Copy link
Member

yeah that looks a bit asymmetric before. Now it is better.

@Dream-Master
Copy link
Member

@wlhlm btw is it possible to add some gui buttons for the super chest/quantum chest as well? or is this not necessary ?

@wlhlm
Copy link
Member Author

wlhlm commented Jun 15, 2023

Probably. Let me check, I haven't looked at those.

@wlhlm
Copy link
Member Author

wlhlm commented Jun 15, 2023

Those don't have a lock widget.

2023-06-15_17 47 49

Copy link
Member

@miozune miozune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh now I remember why I placed widgets like this: It prevented accidentally clicking "void all" button when locking fluid. What do you think? @wlhlm @Dream-Master

@wlhlm
Copy link
Member Author

wlhlm commented Jun 15, 2023

I see, that could be a problem indeed. Is this better?

2023-06-15_18 17 01

@miozune
Copy link
Member

miozune commented Jun 15, 2023

yeah that's better

Also make some labels translatable.
@wlhlm
Copy link
Member Author

wlhlm commented Jun 15, 2023

OK, I've updated the PR with the new voiding button locations

@Dream-Master Dream-Master merged commit 5a81c95 into master Jun 15, 2023
2 checks passed
@Dream-Master Dream-Master deleted the tank-lock-widget-move branch June 15, 2023 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants