Closed
Bug 1929006
Opened 1 year ago
Closed 1 year ago
Accessibility components/Button.js isn't used anywhere
Categories
(DevTools :: Accessibility Tools, task)
DevTools
Accessibility Tools
Tracking
(firefox134 fixed)
RESOLVED
FIXED
134 Branch
| Tracking | Status | |
|---|---|---|
| firefox134 | --- | fixed |
People
(Reporter: nchevobbe, Assigned: nchevobbe)
Details
Attachments
(1 file)
See https://searchfox.org/mozilla-central/search?q=components%2FButton.js&path=&case=false®exp=false
The file isn't loaded from any other places, so we can probably remove it and cleanup the associated CSS rules
| Assignee | ||
Comment 1•1 year ago
|
||
The components weren't used anywhere. We can also remove selectors and rules that were targetting
classes only used in this file.
Updated•1 year ago
|
Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6de8e8b78185
[devtools] Delete unused devtools/client/accessibility/components/Button.js. r=devtools-reviewers,bomsy.
Comment 3•1 year ago
|
||
Backed out for causing devtools failures in browser_accessibility_print_to_json.js.
- Backout link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | devtools/client/accessibility/test/browser/browser_accessibility_print_to_json.js | Uncaught exception in test bound a11YPanelTask - at chrome://mochitests/content/browser/devtools/client/accessibility/test/browser/head.js:563 - TypeError: can't access property "querySelector", row is undefined
Flags: needinfo?(nchevobbe)
| Assignee | ||
Comment 4•1 year ago
|
||
sorry about that, I'm having a look right now
Flags: needinfo?(nchevobbe)
Updated•1 year ago
|
Attachment #9435130 -
Attachment description: Bug 1929006 - [devtools] Delete unused devtools/client/accessibility/components/Button.js. r=#devtools. → Bug 1929006 - [devtools] Delete unused devtools/client/accessibility/components/Button.js. r=#devtools.
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/080d4a9badc5
[devtools] Delete unused devtools/client/accessibility/components/Button.js. r=devtools-reviewers,bomsy.
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox134:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•