Checkboxes in native windows now look like in-content checkboxes
Categories
(Toolkit :: Themes, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox88 | --- | unaffected |
firefox89 | + | verified |
firefox90 | --- | verified |
People
(Reporter: mbalfanz, Assigned: bugzilla)
References
(Regression)
Details
(Keywords: regression, Whiteboard: [proton-modals] [priority:2c] [proton-uplift])
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
Markus pointed out that there are more places where we should show native checkboxes instead of in-content ones.
The places are:
- Clear Recent History dialog
- Page Info -> Permissions panel
- Import Wizard -> Items to Import panel
Updated•3 years ago
|
Comment 1•3 years ago
|
||
It stands to reason that all checkboxes in native-looking windows are affected by this, right? It feels, and maybe I'm wrong, that this bug should not be addressed by going on a hunt for checkboxes in our UI, but to figure out what code change caused the differentiation between in-content checkboxes and native checkboxes to break. For instance, the drop-down menus and buttons also got updated, but they still look native where they should be.
Assignee | ||
Comment 2•3 years ago
|
||
I spoke with Martin and I'm going to take this bug. This was probably regressed by bug 1697315. That bug updated all checkboxes but introduced a [native]
attribute to be used in exceptional cases. Drop-down menus were also fixed with an à-la-carte approach in bug 1697863, since they were also implemented with a [native]
attribute for exceptions.
Assignee | ||
Comment 3•3 years ago
|
||
This patch also ensures we always use the dedicated preferences sanitize.xhtml in about:preferences.
Assignee | ||
Comment 4•3 years ago
|
||
[Tracking Requested - why for this release]:
Inconsistent look in Proton checkboxes, making native dialogs feel out of place. Straightforward patch; easy uplift.
I'll request uplift after I verify the fix in Nightly tomorrow.
Pushed by htwyford@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7057915ac757 Use native checkboxes in all native windows. r=Gijs,preferences-reviewers
Comment 6•3 years ago
|
||
bugherder |
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Comment 7•3 years ago
|
||
Hey harry, just pinging this bug to remind you to request uplift on it as soon as you can (per comment 4)
Assignee | ||
Comment 8•3 years ago
|
||
Comment on attachment 9218990 [details]
Bug 1708209 - Use native checkboxes in all native windows. r?Gijs!
Beta/Release Uplift Approval Request
- User impact if declined: Incorrect look in several native-styled windows, like the Clear Recent History window. Proton regression.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: Please check our various native-styled windows to see that checkboxes are styled like OS checkboxes instead of the checkboxes we use elsewhere in Firefox, such as in about:preferences?
Some of those windows:
- Clear Recent History
- Page info window (lock icon -> connection status -> More Information)
- Setting desktop wallpaper
- Import Wizard -> Items to Import panel
- Unknown content type download window
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): We already support a [native] attribute on checkboxes. This patch just applies that attribute to a few more
<checkbox>
es. - String changes made/needed:
Assignee | ||
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Comment 9•3 years ago
|
||
Comment on attachment 9218990 [details]
Bug 1708209 - Use native checkboxes in all native windows. r?Gijs!
Low risk, approved for 89 beta 8, thanks.
Comment 10•3 years ago
|
||
bugherder uplift |
Comment 11•3 years ago
|
||
This issue is Verified as fixed in our latest Nightly build as well as in Beta 89.0b9 on Windows, Mac and Ubuntu.
However Harry Twyford also mentioned the : Setting desktop wallpaper modal but I couldnt find any checkboxes on it.
Assignee | ||
Comment 12•3 years ago
|
||
(In reply to Rares Doghi from comment #11)
However Harry Twyford also mentioned the : Setting desktop wallpaper modal but I couldnt find any checkboxes on it.
Sorry, I was just listing off native-themed windows. If there are no checkboxes on that window, then we're all good.
Updated•3 years ago
|
Description
•