Closed Bug 1292133 Opened 3 years ago Closed 3 years ago

Inspect element action does not work correctly while RDM is enabled

Categories

(DevTools :: Responsive Design Mode, defect, P1)

51 Branch
defect

Tracking

(firefox48 unaffected, firefox49 unaffected, firefox50 unaffected, firefox51 verified)

VERIFIED FIXED
Firefox 51
Iteration:
51.3 - Sep 19
Tracking Status
firefox48 --- unaffected
firefox49 --- unaffected
firefox50 --- unaffected
firefox51 --- verified

People

(Reporter: mboldan, Assigned: jryans)

References

Details

(Whiteboard: [multiviewport] [reserve-rdm])

Attachments

(2 files)

Attached video 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.
QA Whiteboard: [qe-rdm]
Flags: qe-verify+
Priority: -- → P3
Whiteboard: [multiviewport] [triage] → [multiviewport] [reserve-rdm]
Blocks: 1296498
Assignee: nobody → jryans
Status: NEW → ASSIGNED
Iteration: --- → 51.2 - Aug 29
Priority: P3 → P1
Depends on: 1297569
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 jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/f91a4cdf2718
Add contextmenu tunnelling for inspect element. r=ochameau
Iteration: 51.2 - Aug 29 → 51.3 - Sep 12
https://hg.mozilla.org/mozilla-central/rev/f91a4cdf2718
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
Depends on: 1300103
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
Flags: qe-verify+
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.