Closed Bug 1819372 Opened 1 year ago Closed 1 year ago

Use moz-button-group in ETP panel

Categories

(Toolkit :: UI Widgets, task)

task

Tracking

()

RESOLVED FIXED
114 Branch
Tracking Status
firefox114 --- fixed

People

(Reporter: mstriemer, Assigned: mstriemer)

References

(Blocks 1 open bug)

Details

(Whiteboard: [recomp])

Attachments

(3 files, 1 obsolete file)

The cookie banner reduction section ETP panel has a pair of buttons to change the setting. These should be wrapped in a moz-button-group so that they are ordered correctly. Ensure that the tab order is correct on Windows, particularly.

It's possible that there may need to be an update to moz-button-group to move the light DOM button, since the panel handles the tab key itself. Also check for any existing order: -1 rules that may need updates (likely this one [1])

[1] https://searchfox.org/mozilla-central/rev/aa3ccd258b64abfd4c5ce56c1f512bc7f65b844c/toolkit/themes/shared/popupnotification.css#79

Assignee: nobody → mstriemer
Status: NEW → ASSIGNED
Attachment #9325554 - Attachment is obsolete: true
Attachment #9325555 - Attachment description: WIP: Bug 1819372 - Part 1: Reorder light DOM in moz-button-group r?hjones,tgiles → Bug 1819372 - Part 1: Reorder light DOM in moz-button-group r?hjones,tgiles
Attachment #9325556 - Attachment description: WIP: Bug 1819372 - Part 2: Use moz-button-group in Cookie Banner Reduction panel r?jhirsch,dwalker → Bug 1819372 - Part 2: Use moz-button-group in Cookie Banner Reduction panel r?jhirsch,dwalker
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/39b61e65abfe
Part 1: Reorder light DOM in moz-button-group r=hjones
https://hg.mozilla.org/integration/autoland/rev/faff8ddbf66b
Part 2: Use moz-button-group in Cookie Banner Reduction panel r=pbz
https://hg.mozilla.org/integration/autoland/rev/1983ea2af562
Part 3: Use moz-button-group in ETP report site panel r=pbz
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
Whiteboard: [fidefe-reusable-components] → [recomp]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: