Remove min-widget-size of some menu items.
Categories
(Core :: Widget: Win32, defect)
Tracking
()
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(2 files)
Assignee | ||
Comment 1•2 years ago
|
||
This fixes bug 1799460 for good. You can test this on Win10 with
browser.display.windows.non_native_menus=0 or HCM, but same applies to
Win7/8.
The front-end sets their own sizes anyways, and when the windows sizes
are less than the ones the front-end sets, we see that bug.
This is rather annoying because, even though it's fixable (we can fix
the intrinsic sizing code in the web to deal with this properly), it's
probably not worth fixing, as no web-exposed widget has a min-widget
size other than scrollbars and internal form control parts.
Removing this code altogether just works, since as I said before the
front-end sizes the menu parts anyways, so enforcing a given size in C++
is not necessary.
Comment 3•2 years ago
|
||
bugherder |
Comment 4•2 years ago
|
||
:Emilio, unfortunately, the issue doesn't seem to be fixed. I can reproduce it in both cases, with browser.display.windows.non_native_menus=0 or with high contrast enabled, using the latest Nightly build (20221123094429).
Assignee | ||
Comment 5•2 years ago
|
||
Yes, bug 1801845 will remove all this and make them look non-native.
Comment 6•2 years ago
|
||
Backed out for causing a regression
Assignee | ||
Comment 7•2 years ago
|
||
Let's do this altogether when I do bug 1801845.
Comment 8•2 years ago
|
||
Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/d348127a05b6
Description
•