Menu

#3 jsContextMenu popup callback

open
nobody
5
2008-03-05
2008-03-05
Shatsar
No

i've just commented out these two lines in jsTreeView popupMenu method (line 3157)

var nodo = this.getNodeBySpanId(evt.target.getAttribute('id'));
this.getContextMenu().setEntryEnabled(1, nodo.isEditable());

because they work well with defaultContextMenu, but not with custom Menu.. and isn't the right place to do this check.. maybe we could add a callback method to be implemented in contextMenu instance (like handleResponse in AjaxLoader) automatically called by popup event

Discussion

  • Shatsar

    Shatsar - 2008-03-05
    • summary: jsContextSubmenu popup callback --> jsContextMenu popup callback
     
  • Fabio Sturaro

    Fabio Sturaro - 2008-03-09

    Logged In: YES
    user_id=1962530
    Originator: NO

    I think we can just remove those lines as the callback for a custom contextMenu could be inserted in the popup() method.

     

Log in to post a comment.

MongoDB Logo MongoDB