Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1527 closed defect (wontfix)

PopupMenu2 for non-existent targetId becomes window's context menu.

Reported by: guest Owned by: liucougar
Priority: low Milestone:
Component: Widgets Version: 0.3
Keywords: Cc:
Blocked By: Blocking:

Description

When a PopupMenu2 contextmenu is given a targetNodeIds value that contains an id that is not present in the document the contextmenu is bound as the window's context menu rather than just being ignored for that non-existent targetId.

Change History (6)

comment:1 Changed 13 years ago by bill

Milestone: 0.5
Owner: changed from bill to liucougar

comment:2 Changed 13 years ago by liucougar

Milestone: 0.50.4.1
Status: newassigned

comment:3 Changed 13 years ago by liucougar

Cc: wkeese@… added

Bill:

shall we add a checking in Menu2 ? or users shall ensure the validation of an id?

comment:4 Changed 13 years ago by bill

Cc: wkeese@… removed

I think users should ensure validation of the id, because I don't like increasing dojo's download size just to detect programming errors, but having said that, I don't care much either way. In any case this doesn't seem like a critical bug (not sure why you scheduled it against 0.4.1; I guess because it's trivial to fix)

comment:5 Changed 13 years ago by liucougar

Resolution: wontfix
Status: assignedclosed

I agree

so close this bug as wontfix

please check id before passing it to menu2 bind dom

comment:6 Changed 12 years ago by (none)

Milestone: 0.4.1

Milestone 0.4.1 deleted

Note: See TracTickets for help on using tickets.