Opened 9 years ago

Closed 9 years ago

#11321 closed defect (fixed)

[patch][cla] MenuBar: disabled menu item does not seem "disabled" in high contrast (IE7 & IE8)

Reported by: pilar Owned by: Becky Gibson
Priority: high Milestone: 1.5
Component: Accessibility Version: 1.5.0b2
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

On the MenuBar widget, disabled menu items are displayed the same as the other (enabled) MenuItems. This problem is not in other browsers as Firefox, chrome or Safari.

Attachments (2)

disable_error.JPG​ (11.9 KB) - added by pilar 9 years ago.
11321.patch​ (403 bytes) - added by Becky Gibson 9 years ago.

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by pilar

Attachment: disable_error.JPG​ added

comment:1 Changed 9 years ago by Becky Gibson

Component: a11y - review → Accessibility
Description: modified (diff)
Milestone: tbd → 1.5.1

I assume this is only in high contrast mode as that is what the picture seems to be showing?

comment:2 Changed 9 years ago by pilar

Yes, this problem is using High Contrast mode.

comment:3 Changed 9 years ago by Becky Gibson

Summary: Menu Widget Disable button does not seem "disable" ( IE7 & IE8) → Menu Widget Disable button does not seem "disable" in high contrast( IE7 & IE8)

Changed 9 years ago by Becky Gibson

Attachment: 11321.patch​ added

comment:4 Changed 9 years ago by Becky Gibson

Summary: Menu Widget Disable button does not seem "disable" in high contrast( IE7 & IE8) → [PATCH] Menu Widget Disable button does not seem "disable" in high contrast( IE7 & IE8)

Simple fix is to just add one more style rule for menu bars since the current ones for disabledMenuItems assume the menuItem is is within a TD but in the case of the menubar it is not. Add the following "case" to the style for dijitMenuItemDisabled.

.dj_ie .dijit_a11y .dijitMenuItemDisabled,

I don't see any reason not to include this in 1.5 as it only affects a11y.

comment:5 Changed 9 years ago by bill

Description: modified (diff)
Milestone: 1.5.1 → 1.5
Summary: [PATCH] Menu Widget Disable button does not seem "disable" in high contrast( IE7 & IE8) → [patch][cla] MenuBar: disabled menu item does not seem "disabled" in high contrast (IE7 & IE8)

Sure, looks good to me (although I suspect there are many cases in high-contrast mode where enabled and disabled widgets look the same).

comment:6 Changed 9 years ago by Becky Gibson

Resolution: → fixed
Status: new → closed

(In [22356]) fixes #11321 - needed an additional style definition for disabled menuitems in a menubar

Note: See TracTickets for help on using tickets.