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

StageManager support for fullscreen, always on top and maximized #50

Closed
trixon opened this issue Apr 9, 2022 · 5 comments
Closed

StageManager support for fullscreen, always on top and maximized #50

trixon opened this issue Apr 9, 2022 · 5 comments

Comments

@trixon
Copy link
Contributor

trixon commented Apr 9, 2022

A while back I wrote a similar thing like your stage manager
https://github.com/trixon/almond/blob/3880029726706ab0122de308af2a636860b602a4/util/src/main/java/se/trixon/almond/util/fx/FxHelper.java#L533
but without your screen management and by occupying stage.setOnCloseRequest.

What it does have is support for fullscreen, always on top and maximized.

Do you have any plans for that or should I use part of your code in mine?

@dlemmermann
Copy link
Collaborator

dlemmermann commented Oct 11, 2022 via email

@dlemmermann
Copy link
Collaborator

Wanna contribute that?

@trixon
Copy link
Contributor Author

trixon commented Jan 12, 2023

Sure thing, now that I manage to submit a PR, I'm not really comfortable with that, yet.

@trixon
Copy link
Contributor Author

trixon commented Jan 25, 2023

I'm holding always on top for now, not sure if it's a good idea to have it there, unless one could opt-out from it and the other settings too with some configurator object.

Submitted a PR for the others but it the check failed, is it something I'm supposed to do/fix? @dlemmermann

dlemmermann added a commit that referenced this issue Jan 26, 2023
Add support for maximized and fullscreen in StageManager #50
@dlemmermann
Copy link
Collaborator

The check failed because part of the build is a QA analysis via Sonarcube and I am the only one allowed to call it :-( Something to look into by me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants