39 bytes, text/x-review-board-request
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.
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
Created attachment 8605360 [details] MozReview Request: bz://1156754/pbrosset /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.
4 years ago
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.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Comment on attachment 8605360 [details] MozReview Request: bz://1156754/pbrosset
Created attachment 8620099 [details] MozReview Request: Bug 1156754 - Introduce a pref for the new animation inspector UI
Component: Developer Tools: Inspector → Developer Tools: Animation Inspector
You need to log in before you can comment on or make changes to this bug.