#849926: links wrapped in .contextual-links-wrapper divs are not accessible at all via keyboard alone also problems with screen readers introduced a horizontal scrollbar by positioning .contextual off to the right. While hidden, moving the position right: 0; over 26px for Stark and 19px for Bartik resolves the scrollbar, then when active, returning to it's desired position 0 functions as before.
Admittedly not knowing the intricacies of the accessibility issues of 849926, the upcoming patch resolves the horizontal scrollbar, hopefully not conflicting with the accessibility.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | contextual-scrollbar-1907236-2.patch | 460 bytes | echoz |
| #1 | contextual-scrollbar-1907236-1.patch | 915 bytes | echoz |
| #1 | contextual_before.png | 6.08 KB | echoz |
| #1 | contextual_after.png | 5.46 KB | echoz |
Comments
Comment #1
echoz commentedComment #2
echoz commentedDisregard #1 patch, this results in the same screenshots and functionality.
Using the same selectors, this now toggles the overflow visibility.
Comment #3
mgiffordI can't repeat the error, but at looking at your patch there doesn't seem to be anything it it that would cause barrier to someone to use the patch.
Patch seems to apply nicely.
Comment #4
echoz commentedThe horizontal scrollbar is still there, but is not on admin screens. It is present when there is a full width contextual links wrapper that expands to the right side of the screen. I see it on non admin screens of Stark and Bartik.
Comment #5
echoz commentedresolved by #1874664: Introduce toolbar level "Edit" mode that shows all contextual links
Comment #6
mgiffordCool, thanks for closing this issue (and identifying it)!