Closed
Bug 785186
Opened 11 years ago
Closed 11 years ago
Use the context menu in the markup panel
Categories
(DevTools :: Inspector, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 17
People
(Reporter: dcamp, Unassigned)
References
Details
(Whiteboard: [fixed-in-fx-team])
Attachments
(1 file)
6.31 KB,
patch
|
jwalker
:
review+
|
Details | Diff | Splinter Review |
This re-enables the context menu in the new markup panel. Things get a little tricky because the markup panel and inspector don't necessarily agree on the selection (See bug 785180).
Attachment #654733 -
Flags: review?(jwalker)
Comment 2•11 years ago
|
||
Comment on attachment 654733 [details] [diff] [review] v1 Review of attachment 654733 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/devtools/highlighter/inspector.jsm @@ +1203,5 @@ > break; > } > }, > > + // Return the currently-selected node for the purposes of the If we care about comment style, then we might /**...*/ this.
Attachment #654733 -
Flags: review?(jwalker) → review+
Reporter | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/9b58a0a085ee
Reporter | ||
Updated•11 years ago
|
Whiteboard: [fixed-in-fx-team]
Reporter | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9b58a0a085ee
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Updated•5 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•