Changes between Initial Version and Version 2 of Ticket #8603


Ignore:
Timestamp:
Feb 9, 2009, 7:39:42 PM (11 years ago)
Author:
bill
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #8603 – Description

    initial v2  
    66  - a contextmenu event
    77
    8 Selector.js has this code:
     8The actual failure point is in Selector.js in this code:
    99
    1010{{{
     
    1616}}}
    1717
    18 The above code should be checking if e.button == 1
     18The above code shouldn't be running if e.button == 2.   Actually, the whole function shouldn't be running if e.button == 2, and probably some other functions too.
     19
     20Note that on IE, a left-click has e.button == 1, and on FF3/safari e.button == 0, so be careful with how you check.
    1921
    2022Due to an apparent firefox bug (or perhaps this is correct behavior?), canceling the right-mousedown event also cancels the context-menu event.  Thus, this bug manifests itself by breaking context menus on draggable nodes.   (Again though, it only happens on FF3/mac when using a real mouse.)
    2123
    22 The test case is in #8388.   (Try right clicking nodes in the bottom right tree.)  Although that test case is against the tree DnD code, the problem exists in the generic DnD code as well.
     24The test case is attached to #8388.   (Try right clicking nodes in the bottom right tree.)  Although that test case is against the tree DnD code, the problem exists in the generic DnD code as well.
    2325
    2426(Derived from #8388)