-
-
Notifications
You must be signed in to change notification settings - Fork 10.5k
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
Main menu bar isn't affected by DisplaySafeAreaPadding #1439
Comments
Hello Louis, I think a PR is coming for that and I will look at it. My concern is/was that the padding within the menu bar will almost necessarily always look asymmetrical in that situation. Out of curiosity, which sorts of value and/or range do you typically use for safe area padding? And what’s the font size? |
Ought to clarify: the reason the asymmetry is problematic is because we have a non-zero default value of ImVec2(4,4) for |
… x position. Fixes main menu bar problems with TVs. (#1439)
…splaySafeAreaPadding defaults to (3,3) instead of (4.4). Comments, tab to spaces, Changelog. (#1439)
Louis, this should be fixed now (see 2 commits above). I also made a third commit to improve handling of the safe area for popups/tooltips/menus. In the future viewport branch this setting will evolve a little, as safe area may be defined on a per-monitor basis. I'm not touching it right now because I don't have much visibility on this but ideally viewports could be used across two machines. |
@ldecarufel Are you still using Tagging @dougbinks as I vaguely remember (might be wrong) that you allow used safe area? (EDIT: well yes, you submitted part of it :) (EDIT: Basically I am trying to figure out if nowadays it would be reasonable to drop SafeArea padding support) |
I know of no way on PC Windows to set the safe area for a TV, so I would have thought this was still required. |
The main menu bar ignores the DisplaySafeAreaPadding style and is always drawn at (0,0).
On TVs with overscan, the main menu can be unusable because it is drawn above the visible area.
The text was updated successfully, but these errors were encountered: