Opened 9 years ago
Closed 9 years ago
#17045 closed defect (fixed)
dijit/PopupMenuItem baseClass
Reported by: | Martin Repta | Owned by: | bill |
---|---|---|---|
Priority: | undecided | Milestone: | 1.10 |
Component: | Dijit | Version: | 1.8.3 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
dijit/PopupMenuItem should have custom baseClass defined, currently it has none.
patch file attached.
Attachments (1)
Change History (4)
Changed 9 years ago by
Attachment: | PopupMenuItem.patch added |
---|
comment:1 Changed 9 years ago by
Milestone: | tbd → 1.10 |
---|
comment:2 Changed 9 years ago by
Strangely though CheckedMenuItem? resets the base class from dijitMenuItem to dijitCheckedMenuItem, which means that it too apparently isn't picking up rules like:
.dijitMenuItemDisabled * { /* for a disabled menu item, just set it to mostly transparent */ opacity:0.5; cursor:default; }
Probably both of them should have a dual base class.
comment:3 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Well, presumably PopupMenuItem currently inherits its baseClass from MenuItem. Looks like your patch would *change* the baseclass from dijitMenuItem to dijitPopupMenuItem, which would probably break lots of things, but maybe we can add a dijitPopupMenuItem base class.