Closed Bug 1576652 Opened 5 years ago Closed 5 years ago

Remove unused state in Accordion component

Categories

(DevTools :: Debugger, task)

task
Not set
normal

Tracking

(firefox70 fixed)

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: fvsch, Assigned: fvsch)

Details

Attachments

(1 file)

The devtools/client/debugger/src/components/shared/Accordion.js component defines some state but never uses it. In particular, the 'opened' status of items comes from props, and the copy in component state is not read:

this.state = {
  opened: props.items.map(item => item.opened),
  created: [],
};
Assignee: nobody → florens
Status: NEW → ASSIGNED
Pushed by florens@fvsch.com: https://hg.mozilla.org/integration/autoland/rev/3b11b55b1c97 Remove unused state in Debugger's Accordion component; r=jlast
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: