-
Notifications
You must be signed in to change notification settings - Fork 117
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
Make popup windows more consistent #1716
Conversation
Failing CI |
4c40128
to
d3ddfea
Compare
Fixes #1733 |
@@ -179,7 +179,7 @@ public void initialize() { | |||
} | |||
|
|||
public void onCloseButtonClicked() { | |||
onCloseButtonClickedListener.run(); | |||
getRoot().setVisible(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think that is sufficent to effectively close it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously this listener was created when the JFXDialog was created. I guess it was needed to make the dialog disappear properly / to trigger the close animation.
Now it just hides and reveals the create game window. It works in the same way in the mod and map details.
If you want to have it "more" closed I need you to go into more detail how you want to achieve that.
Don't know if it's worth the effort just to unload a few elements and one map preview picture from memory, though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use JfxDialog
We wanna use jfx on vault not drop jfx in create game dialog |
outdated and no longer needed |
Brings the style of the popup windows (map, mod, and replay detail and create game) more in line with each other and fixes minor UI Issues.
All windows keep sensible min and max sizes and hover in the middle of the screen.
The Create Game window now has a proper cancel button and no superfluous title anymore
"Min" is now in front of the text field instead of behind
Map detail buttons are lower so they don't crash with the map name
"Description" removed
Mod detail install button is now a highlighted button