Created attachment 8777797 [details] inspect element.mov [Affected versions]: - Firefox 51.0a1 (2016-08-03) [Affected platforms]: - Windows 10 x64, Mac OS X 10.11.1, Ubuntu 16.04 x64 [Steps to reproduce]: 1. Launch Firefox. 2. From about:config, enable the devtools.responsive.html.enabled pref. 3. Open RDM. 4. Enable Toggle tools. 5. Right click on an element from the page and select the 'Inspect Element' option. [Expected result]: - The details of the selected element are displayed in the Inspector. [Actual result]: - The inspected element is not selected in the HTML tree (see the attached screencast). [Regression range]: - This is not a regression.
Priority: -- → P3
Whiteboard: [multiviewport] [triage] → [multiviewport] [reserve-rdm]
status-firefox51: affected → fix-optional
Assignee: nobody → jryans
Status: NEW → ASSIGNED
This work depends on the change in bug 1297569, so you'll need to apply that patch as well (if you want to test during review).
Comment on attachment 8785343 [details] Bug 1292133 - Add contextmenu tunnelling for inspect element. https://reviewboard.mozilla.org/r/74584/#review72788 I really appreciate such patch to review after having to review patches to like bug 1297362 ;) Note that Inspect element still appear when right clicking within RDM document and fails with "Permission denied to access property "ownerDocument"" you might want to disable the context menu entry or something?
Attachment #8785343 - Flags: review?(poirot.alex) → review+
Comment on attachment 8785343 [details] Bug 1292133 - Add contextmenu tunnelling for inspect element. https://reviewboard.mozilla.org/r/74584/#review72788 > Note that Inspect element still appear when right clicking within RDM document and fails with "Permission denied to access property "ownerDocument"" you might want to disable the context menu entry or something? Hmm, thanks for pointing this out! I think I'll defer this to bug 1278762 (the more general "context menu doesn't really work" bug). There isn't really a simple way to disable just this one item in this specific context. Over in that bug, I might make my own context menu with just the few things that work...? Unsure so far.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/f91a4cdf2718 Add contextmenu tunnelling for inspect element. r=ochameau
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox51: fix-optional → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
The issue is no longer reproducible on Firefox 51.0a1 (2016-09-13), using the STR from Comment 0. The tests were performed under Windows 10x64, Mac OS X 10.11.1 and under Ubuntu 16.04x64. I am marking this issue Verified Fixed.
Status: RESOLVED → VERIFIED
status-firefox51: fixed → verified
You need to log in before you can comment on or make changes to this bug.