287621 - Right click (context menu) in source view margin should not change breakpoint

Status: RESOLVED FIXED

(Other Applications Graveyard :: Venkman JS Debugger, defect)

Description

[Alex Vincent [:WeirdAl]]

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.

Comment 1

[Alex Vincent [:WeirdAl]]

Attachment: patch

Comment 2

[Robert Ginda]

Comment on attachment 178527 [details] [diff] [review]
patch

r=rginda

Comment 3

[Alex Vincent [:WeirdAl]]

I'll need someone to check this in, please.

Comment 4

[Alex Vincent [:WeirdAl]]

checked in by timeless :)