Opened 11 years ago
Closed 11 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 )
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)
Change History (8)
Changed 11 years ago by
Attachment: | disable_error.JPG added |
---|
comment:1 Changed 11 years ago by
Component: | a11y - review → Accessibility |
---|---|
Description: | modified (diff) |
Milestone: | tbd → 1.5.1 |
comment:3 Changed 11 years ago by
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 11 years ago by
Attachment: | 11321.patch added |
---|
comment:4 Changed 11 years ago by
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 11 years ago by
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 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
I assume this is only in high contrast mode as that is what the picture seems to be showing?