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

MacOS: Menubar is on dialog when it should be at top of screen #34

Open
Azhrei opened this issue Mar 31, 2019 · 4 comments
Open

MacOS: Menubar is on dialog when it should be at top of screen #34

Azhrei opened this issue Mar 31, 2019 · 4 comments
Labels

Comments

@Azhrei
Copy link
Member

Azhrei commented Mar 31, 2019

Describe the bug
The Swing MenuBar is anchored to the top of the window by default, but on macOS is should be attached to the top of the screen.

To Reproduce
Steps to reproduce the behavior:

  1. Open the application on a Mac
  2. Notice where the menu appears.

Expected behavior
Menu should be at the top of the screen.

TokenTool Info

  • Version: 2.1-rc.3

Desktop (please complete the following information):

  • OS: macOS
  • Version 10.14 (tested, but applies to all versions)

Additional context
MapTool has similar code, I believe in AppMenuBar.java

@JamzTheMan
Copy link
Member

Looks like there is a useSystemMenuBarProperty. Would have to add that to the controller init as we use fxml.

Even with that, it still is a little funky I guess from what i am reading. Not sure if it's better with Java 10 or 11. Be curious how it works with this using 11...

@Phergus Phergus added the bug label Jul 12, 2021
@Phergus
Copy link
Contributor

Phergus commented Jul 12, 2021

Need confirmation that this is still happening with TT 2.2.

@Azhrei
Copy link
Member Author

Azhrei commented Jul 14, 2021

This is still happening.

I remember there were a few things different about the menu bar in MapTool — the position of the About and Quit menu options come to mind, and I think the menubar property. Comparing the code against MT is probably the way to go.

@Phergus
Copy link
Contributor

Phergus commented Jul 14, 2021

Too bad.

Just a note that TT is straight JavaFX for the UI as far as I know so Jamz's comment about useSystemMenuBarProperty may be the fix.

https://openjfx.io/javadoc/14/javafx.controls/javafx/scene/control/MenuBar.html

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

No branches or pull requests

3 participants