Closed
Bug 785186
Opened 12 years ago
Closed 12 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•12 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•12 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/9b58a0a085ee
|
|
Reporter | |
Updated•12 years ago
|
Whiteboard: [fixed-in-fx-team]
|
|
Reporter | |
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9b58a0a085ee
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
|
||
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•