Closed Bug 1156754 Opened 6 years ago Closed 6 years ago

Pref-off the new animation-inspector UI v3 by default

Categories

(DevTools :: Inspector: Animations, defect)

defect
Not set
normal

Tracking

(firefox41 fixed)

RESOLVED FIXED
Firefox 41
Tracking Status
firefox41 --- fixed

People

(Reporter: pbro, Assigned: pbro)

References

Details

Attachments

(1 file, 1 obsolete file)

Introduce a pref that will be used to develop the animation-inspector UI v3 (bug 1153271) while keeping the current UI as is.
I doubt that the new UI will be fully functional after one bug fix, so this pref would help smooth out the transition.
Blocks: 1153271
Blocks: 1155663
Assignee: nobody → pbrosset
The patch I'll attach here really just adds a pref to firefox.js, something along the lines of newUI=true/false.

The strategy here is that the new UI we want to put in place in bug 1153271 is sufficiently different from the current one that we can't migrate from one to the other, it has to be a switch over at some stage.

This pref will make it possible to both:
- develop the new UI hidden, allowing people to still use the current one, and try out the few bits of the new UI that will land if they want to
- create tests for the new UI, while making sure the existing ones still work.
Status: NEW → ASSIGNED
/r/8697 - Bug 1156754 - Introduce a pref for the new animation inspector UI

Pull down this commit:

hg pull -r a510d1741570e51517672e9f054979709802b277 https://reviewboard-hg.mozilla.org/gecko/
Attachment #8605360 - Flags: review?(jwalker)
Joe, I wasn't sure who to ask for review for this (small) patch.
Let me know if you think the approach in comment 1 makes sense.
Attachment #8605360 - Flags: review?(jwalker) → review+
(In reply to Patrick Brosset [:pbrosset] [:patrick] from comment #3)
> Joe, I wasn't sure who to ask for review for this (small) patch.
> Let me know if you think the approach in comment 1 makes sense.

Yes makes sense to me.
https://hg.mozilla.org/mozilla-central/rev/1ef7c8593ae6
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Attachment #8605360 - Attachment is obsolete: true
Attachment #8620099 - Flags: review+
Component: Developer Tools: Inspector → Developer Tools: Animation Inspector
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.