Closed
Bug 1982964
Opened 9 months ago
Closed 8 months ago
Update moz-box-* elements so they can handle wrappers between the group and children
Categories
(Toolkit :: UI Widgets, enhancement)
Toolkit
UI Widgets
Tracking
()
RESOLVED
FIXED
144 Branch
| Tracking | Status | |
|---|---|---|
| firefox144 | --- | fixed |
People
(Reporter: mstriemer, Assigned: mstriemer)
References
(Blocks 1 open bug)
Details
(Whiteboard: [recomp])
Attachments
(2 files)
In SRD 2025 we're going to want to have setting-control wrappers around moz-box-items (of any variety). This unfortunately breaks the styling since the box-group is trying to override the styling of immediate children, and the box item is no longer the immediate child.
Find a way to get the styling to continue to work, it can be a little hacky, using a @container style() query seems like it would be pretty clean, but unfortunately we don't support that right now.
/* Example code for some follow up later maybe? Doesn't work in 143 */
/* moz-box-common.css */
:host {
@container style(--box-type: middle) or style(--box-type: last) {
--box-border-radius-start: 0;
}
@container style(--box-type: first) or style(--box-type: middle) {
--box-border-radius-end: 0;
--box-border-width-end: 0;
}
}
/* moz-box-group.css */
::slotted(*) {
--box-type: middle;
}
::slotted(*:first-child) {
--box-type: first;
}
::slotted(*:last-child) {
--box-type: last;
}
| Assignee | ||
Updated•9 months ago
|
Whiteboard: [recomp]
Updated•9 months ago
|
| Assignee | ||
Comment 1•9 months ago
|
||
Updated•9 months ago
|
Assignee: nobody → mstriemer
Status: NEW → ASSIGNED
Updated•9 months ago
|
Attachment #9509565 -
Attachment description: Bug 1982964 - Update moz-box-group/item to work with config-based prefs r?#recomp-reviewers → Bug 1982964 - Part 1: Handle slotting non-MozBaseInput elements in config-based settings r?#recomp-reviewers!
Comment 2•9 months ago
|
||
Pushed by hjones@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/4e56e93dca55
https://hg.mozilla.org/integration/autoland/rev/3538d541aa1f
Part 1: Handle slotting non-MozBaseInput elements in config-based settings r=tgiles
https://github.com/mozilla-firefox/firefox/commit/fc8112251e54
https://hg.mozilla.org/integration/autoland/rev/1b3103d6f570
Part 2: update moz-box-group to handle hidden elements and non-moz-box-* elements r=desktop-theme-reviewers,jules,mstriemer
Comment 4•8 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/3538d541aa1f
https://hg.mozilla.org/mozilla-central/rev/1b3103d6f570
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
status-firefox144:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 144 Branch
Updated•8 months ago
|
QA Whiteboard: [qa-triage-done-c145/b144]
You need to log in
before you can comment on or make changes to this bug.
Description
•