MenuItem: can't change disabled color w/out using !important

[haysmark]

In dijit.css, you see these lines that get applied to all themes:

.dj_ie .dijit_a11y .dijitMenuItemDisabled,
.dj_ie .dijit_a11y .dijitMenuItemDisabled td,
.dj_ie .dijitMenuItemDisabled *,
.dj_ie .dijitMenuItemDisabled td {
    color:gray !important;
    filter: alpha(opacity=35);
}

The problem is that custom themes inheriting the dijit.css infrastructure can't changed the disabled color.

[bill]

Agreed in principle: the themes can override the color but they need to use !important, which we try not to do in theme files. I imagine the !important is there to:

• solve a precedence issue to make the gray override the color of claro (etc.) enabled MenuItems
• for dijit_a11y to override theme settings

Do you happen to have a patch to fix this?

[bill]

In [29599]:

Remove !important for MenuItem CSS, IE is working without it (both for disabled MenuItem's and disabled MenuBarItem's), fixes #15875 on trunk.

[bill]

In [29600]:

Remove !important for MenuItem CSS, IE is working without it (both for disabled MenuItem's and disabled MenuBarItem's), fixes #15875 on 1.8 branch.