The admin/modules page has table headers whose text overlaps. The left-most column has the word "ENABLED" as its column header, but that is supposed to be invisible text. Instead it is overlapping the next column header, making both illegible. Screenshot attached from Chrome 28. Also verified in Safari 6.
The problem is caused by use of the CSS class "element-invisible" which is no longer part of D8. That class has been replaced with "visually-hidden". This is a follow-up fix to #1990544: Convert system_modules() to a Controller which used the CSS class "element-invisible".
This will be an easy one-line fix.
Comment | File | Size | Author |
---|---|---|---|
#4 | fixed.png | 6.04 KB | xjm |
#1 | 2046301-ModulesListFormAccessibilityStyleFix-2.patch | 1.09 KB | bowersox |
#1 | Screen Shot 2013-07-19 at 9.12.53 PM.png | 153.77 KB | bowersox |
Screen Shot 2013-07-19 at 1.03.13 PM.png | 132.98 KB | bowersox |
Comments
Comment #1
bowersox CreditAttribution: bowersox commented1-line patch attached. It fixed the CSS class so the header goes back to being invisible, and no text overlaps. A screenshot is also attached (showing the correct, patched display).
Please review.
Comment #2
tim.plunkettClosed #2046327: CSS regression in table header on admin/modules ("Enabled" header not hidden) as a dupe. Let's get this in before more issues get filed.
Comment #3
xjmComment #4
xjmIt works:
Comment #5
hass CreditAttribution: hass commentedPer https://drupal.org/node/1363112#comment-7665241 cnw
Comment #6
bowersox CreditAttribution: bowersox commentedWe might want to get this patch in place in order to fix this visual overlap bag. If the CSS class name change happens in that other issue, there would be a global search-and-replace that would change the name used here to whatever the new name will be. But in the mean time I don't think we want to leave the D8 codebase in this inconsistent state with a bug that everyone who uses the Modules page will see. My vote is for RTBC on this simple fix.
Comment #7
xjm@hass, no, it is not NW until that issue has consensus. This is a bugfix.
Comment #8
alexpottCommitted 96868e9 and pushed to 8.x. Thanks!