Closed Bug 1929006 Opened 1 year ago Closed 1 year ago

Accessibility components/Button.js isn't used anywhere

Categories

(DevTools :: Accessibility Tools, task)

task

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&regexp=false
The file isn't loaded from any other places, so we can probably remove it and cleanup the associated CSS rules

The components weren't used anywhere. We can also remove selectors and rules that were targetting
classes only used in this file.

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.

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)

sorry about that, I'm having a look right now

Flags: needinfo?(nchevobbe)
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
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: