Community
Participate
Working Groups
The Theme Preview on the IDE Theme settings page needs accessibility work. There are duplicate ids, landmarks that shouldn't be there, and the whole thing (from the point of view of a screen reader) needs to be marked up as basically an "interactive" image that shows a preview of the theme.
GitHub Pull Request 285 created by [carmacleod] https://github.com/eclipse/orion.client/pull/285
Fixed in PR in comment 1. Notes: - made ids unique (deleted those that are not used in preview) - removed unused elements (i.e. header, file input, slideout panel, dropdown menu panels) - added missing labels - localized all label strings - removed semantics that aren't appropriate in preview (i.e. nav, status) - updated menu semantics to match current edit page menus - updated runBar buttons and links to match current edit page - added desktopmode style to tree to match current - marked up entire preview as an 'interactive' image within a region - changed settings.css @import from "../orion/editor/editor.css" (which was an invalid path) to "../edit/edit.css"; (which is needed for the runbar styles) Note: deleted redundant auxpane id, but needed #auxpane style, so changed rule to .auxpane:not(#leftPane) (because elements that relied on auxpane id also have auxpane class, but leftPane on git page has auxpane class and doesn't want #auxpane style)
