#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 15 years ago by
Milestone: | → 0.5 |
---|---|
Owner: | changed from bill to liucougar |
comment:2 Changed 14 years ago by
Milestone: | 0.5 → 0.4.1 |
---|---|
Status: | new → assigned |
comment:3 Changed 14 years ago by
Cc: | [email protected]… added |
---|
comment:4 Changed 14 years ago by
Cc: | [email protected]… 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 14 years ago by
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
I agree
so close this bug as wontfix
please check id before passing it to menu2 bind dom
Note: See
TracTickets for help on using
tickets.
Bill:
shall we add a checking in Menu2 ? or users shall ensure the validation of an id?