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

Fix toolbar overlaying frames #324

Merged
merged 1 commit into from Jul 8, 2019
Merged

Conversation

Meorawr
Copy link
Member

@Meorawr Meorawr commented Jul 7, 2019

Fixes #324.

"Fullscreen" frames like the world map when maximized don't set the frame strata to FULLSCREEN, which I guess is what used to happen pre-world map rework. Instead they just bump up their frame levels, which is why opening things like the talent panel will show it above the world map.

The best fix for this is to set the strata of the toolbar down to MEDIUM, which will cause it to no longer overlay such panels. This also fixes the issue whereby we'd overlay our own TRP main window with the toolbar if maximized.

"Fullscreen" frames like the world map when maximized don't set
the frame strata to FULLSCREEN, which I guess is what used to
happen pre-world map rework. Instead they just bump up their
frame levels, which is why opening things like the talent panel
will show it above the world map.

The best fix for this is to set the strata of the toolbar down to
MEDIUM, which will cause it to no longer overlay such panels. This
also fixes the issue whereby we'd overlay our own TRP main window
with the toolbar if maximized.
@Meorawr Meorawr requested a review from Solanya July 7, 2019 21:40
@Ellypse Ellypse merged commit fee9e58 into development Jul 8, 2019
@Meorawr Meorawr deleted the bugfix/toolbar-frame-strata branch July 8, 2019 16:36
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

Successfully merging this pull request may close these issues.

None yet

2 participants