Opened 12 years ago

Closed 12 years ago

#2310 closed defect (fixed)

[cla] menu2 references unavailable method closeAll()

Reported by: mikewse@… Owned by: liucougar
Priority: high Milestone: 0.9
Component: Widgets Version: 0.4.1
Keywords: Cc:
Blocked By: Blocking:

Description

In Menu2.js svn 6973 from 1/3/07, there is:

  478: this.parent.closeAll(true);

which is probably a remainder from PopupContainer?.

It breaks clicking an item without a submenu in the menubar, see f ex the "Click Me" item in testswidget est_Menu2.html.

Change History (3)

comment:1 Changed 12 years ago by joose

  • Revision: 7065
  • Error msg: Menu2.js: line 468: this.parent.closeAll is not a function

Happens atleast when you just click around in menubar. Can be replicated atleast with FF and unit tests.

comment:2 Changed 12 years ago by liucougar

Milestone: 0.9
Owner: changed from bill to liucougar

comment:3 Changed 12 years ago by bill

Resolution: fixed
Status: newclosed

Fixed by Menu/PopupManager? rearchitecture.

Note: See TracTickets for help on using tickets.