Closed
Bug 287621
Opened 19 years ago
Closed 19 years ago
Right click (context menu) in source view margin should not change breakpoint
Categories
(Other Applications Graveyard :: Venkman JS Debugger, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: WeirdAl, Assigned: WeirdAl)
References
()
Details
Attachments
(1 file)
939 bytes,
patch
|
rginda
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
When I'm trying to adjust the properties of a breakpoint, a right-click will cause onMarginClick to act, changing a current breakpoint to a future breakpoint, and/or clearing a future breakpoint. Steps to reproduce: (1) Open a script to view in the Source Code panel. (2) Set a current breakpoint through the Source Code panel. (3) Right-click on the breakpoint through the Source Code panel. Expected results: A context menu for the current breakpoint. Actual results: A context menu for a future breakpoint. All we have to do is add a check for (event.button == 0). In that case, it is a left-button mouse click.
Assignee | ||
Comment 1•19 years ago
|
||
Assignee: rginda → ajvincent
Status: NEW → ASSIGNED
Assignee | ||
Updated•19 years ago
|
Attachment #178527 -
Flags: review?(rginda)
Comment 2•19 years ago
|
||
Comment on attachment 178527 [details] [diff] [review] patch r=rginda
Attachment #178527 -
Flags: review?(rginda) → review+
Assignee | ||
Updated•19 years ago
|
Attachment #178527 -
Flags: superreview?(bzbarsky)
Updated•19 years ago
|
Attachment #178527 -
Flags: superreview?(bzbarsky) → superreview+
Assignee | ||
Comment 3•19 years ago
|
||
I'll need someone to check this in, please.
Assignee | ||
Comment 4•19 years ago
|
||
checked in by timeless :)
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•