Closed Bug 1305416 Opened 3 years ago Closed 3 years ago
Shift+F2 no longer closes Developer Toolbar in 49
.0 .1, broken hotkey
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0 Build ID: 20160922113459 Steps to reproduce: - Use Firefox Version 49.0.1 - open the Developer Toolbar (Shift+F2) - try to close the Developer Toolbar by pressing Shift+F2 again Actual results: The Developer Toolbar does not close and the hotkey has no function. Expected results: The Developer Toolbar should have been toggled when pressing Shift+F2. When it's open, it should close, and when it's closed, it should open. It used to toggle in all the previous versions, but now the only way to close it is by clicking the tiny X with the mouse cursor.
Status: UNCONFIRMED → NEW
Component: Untriaged → Developer Tools: Graphic Commandline and Toolbar
Ever confirmed: true
Regression window: https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=f4fbac42bc5e86aeb36fb2d4157fa6dbcf4e441e&tochange=be6fddfc1390a14cd560eb843162f0cccd804e63 Regressed by: be6fddfc1390 James Long — Bug 1265854 - replace uses of nsiDOMNode constants in devtools frontend r=tromey
Whiteboard: [devtools-html] [triage]
Error shows Browser Console: TypeError: nodeConstants is undefined[Learn More] developer-toolbar.js:400:1
Assignee: nobody → jlong
Status: NEW → ASSIGNED
Iteration: --- → 52.1 - Oct 3
Priority: -- → P1
QA Contact: cristian.comorasu
Whiteboard: [devtools-html] [triage] → [reserve-html]
Tracked for 50+ since it's a new regression since Fx49.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/fx-team/rev/9c71ad1d8a69 fix closing the developer toolbar with keyboard shortcut r=me
Comment on attachment 8795439 [details] [diff] [review] 1305416.patch Approval Request Comment [Feature/regressing bug #]: Bug 1265854 [User impact if declined]: users can't close gcli as expected [Describe test coverage new/current, TreeHerder]: NA [Risks and why]: It's an extremely small change that only affects part of the devtools [String/UUID change made/needed]:
Hello there, could you please verify this issue is fixed as expected on a latest Nightly build (09-30)? Thanks!
(In reply to Ritu Kothari (:ritu) from comment #9) > Hello there, could you please verify this issue is fixed as expected on a > latest Nightly build (09-30)? Thanks! Hello, the issue is fixed as expected on 52.0a1 (2016-09-29) (64-bit), thank you very much!
Comment on attachment 8795439 [details] [diff] [review] 1305416.patch Fixes a recent regression, Aurora51+, Beta50+
Hi James, if you tested this fix manually then instead of saying NA in test coverage section of uplift template, please mention that you tested it manually in future. Generally we do not want to uplift any fixes that haven't either been verified manually/externally or via automation. Thanks!
(In reply to Ritu Kothari (:ritu) from comment #12) > Hi James, if you tested this fix manually then instead of saying NA in test > coverage section of uplift template, please mention that you tested it > manually in future. Generally we do not want to uplift any fixes that > haven't either been verified manually/externally or via automation. Thanks! Gotcha, will do, thanks.
I was able to reproduce the issue using Firefox 49.0 RC on Windows 10 Pro x64. The fix was verified on latest Nightly 52.0a1 (2016-09-29), latest Aurora 51.0a2 (2016-09-30) and Beta 50.0b3 Build 1, across platforms Windows 10 Pro x64, Ubuntu 16.04 LTS x64 and Mac OS Yosemite 10.10.4
I reproduced this bug using Fx 52.0a1, build ID: 20160926030203, on Windows 10 x64. I can confirm this issue is fixed, I verified using: Fx50.0b3 (20160929120120) Fx51.0a2 (20160930004005) Fx52.0a1 (20160930030315) On the following platforms: Windows 10 x64 Mac OS X 10.10.5 Ubuntu 14.04 LTS Cheers!
You need to log in before you can comment on or make changes to this bug.