Make preferences use modern flexbox emulation.
Categories
(Firefox :: Settings UI, task, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox107 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Regressed 4 open bugs)
Details
Attachments
(2 files, 1 obsolete file)
Assignee | ||
Comment 1•2 years ago
|
||
Emulated flexbox doesn't support this, but allows you to specify widths
in percentages instead, so do that.
Assignee | ||
Comment 2•2 years ago
|
||
This shouldn't have any observable behavior change, but this is the
first page that uses modern flexbox layout instead of XUL layout.
In practice, this means that various interesting bugs that arise from
poor interaction between xul and block / flex / grid should disappear /
no longer be an issue for front-end developers.
Depends on D157073
Assignee | ||
Comment 3•2 years ago
|
||
This shouldn't have any observable behavior change, but this is the
first page that uses modern flexbox layout instead of XUL layout.
In practice, this means that various interesting bugs that arise from
poor interaction between xul and block / flex / grid should disappear /
no longer be an issue for front-end developers.
Note that this depends on bug 1790303.
Comment 4•2 years ago
|
||
Comment on attachment 9294176 [details]
Bug 1790307 - Use modern flexbox in about:addons. r=#extension-reviewers
Revision D157075 was moved to bug 1790308. Setting attachment 9294176 [details] to obsolete.
Updated•2 years ago
|
Comment 6•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/646ae1f6d62c
https://hg.mozilla.org/mozilla-central/rev/fd57a3ea2b88
Comment 7•2 years ago
|
||
Emilio, I know you've put up patches for a bunch of the regressions but given soft freeze has now started, should we consider backing out the second patch here and relanding it after soft freeze to give ourselves some time to address any fallout, discovered or otherwise?
Assignee | ||
Comment 8•2 years ago
|
||
Yeah. Most fixes are one-liners, not super-concerning, but that's fair.
We'd need to back out both patches tho, otherwise the applications menu would be broken (because width: 50% doesn't work on XUL)... Would it be reasonable to back both out from beta as soon as the merge happens? Alternatively I guess we can back them out today, and I can reland after the soft freeze, if you want.
Comment 9•2 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #8)
Would it be reasonable to back both out from beta as soon as the merge happens? Alternatively I guess we can back them out today, and I can reland after the soft freeze, if you want.
I think backing out today and relanding on Monday/Tuesday is the easiest in that you don't need to request approval for anything (sheriffs will back you out on request), but I don't have a super strong preference either way.
Comment 10•2 years ago
|
||
Backed out as per Emilio's request.
Updated•2 years ago
|
Assignee | ||
Comment 11•2 years ago
|
||
I asked sheriffs to back out, will re-land on monday after the freeze is over.
Comment 12•2 years ago
|
||
Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/f848304d50f3
Comment 13•2 years ago
|
||
backout bugherder |
Comment 14•2 years ago
|
||
Comment 15•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/fa16221c24fe
https://hg.mozilla.org/mozilla-central/rev/0bd09bf30151
Description
•