Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I couldn't find any classes/IDs to use to make the list styles breaking the vertical tabs in #1 more specific. I tried to change the styles in #1 to only target 'Frontend Bartik' but not kick in on 'Admin theme' Bartik and there is nothing available.
I agree that adding an overriding selector and styles to fix this is easiest option here, seeing as Bartik itself is causing the issue.
The patch still applies cleanly, the code standards are fine and it fixes the issue. So I'm setting this to RTBC.
Comment | File | Size | Author |
---|---|---|---|
#3 | bartik_vertical_tabs_styles-2003766-3.patch | 431 bytes | InternetDevels |
#3 | bartic-node-edit-vertical-tabs.png | 29.74 KB | InternetDevels |
Comments
Comment #1
KeyboardCowboyThe specific issue is that there are 2 CSS declarations in core/themes/bartik.css/style.css that are overriding core/misc/vertical-tabs.css
are both overriding
Attached is a patch to redefine ul.vertical-tabs-list with the proper margin and padding, though I believe the best fix is to not override it in the first place. Since these are such wide sweeping declarations I'm hesitant to start altering them without some additional input.
Comment #2
Anonymous (not verified) CreditAttribution: Anonymous commentedStill present in alpha 7.
Comment #3
InternetDevels CreditAttribution: InternetDevels commentedHi, previous patch can't be applied. Attached another patch ;)
Comment #4
emma.mariaAs CSS in Bartik is causing this issue because of the top open selectors stated in #1 the reset added in #3 seems to be the easiest solution that fixes the issue with only
Comment #5
alexpottCommitted d98e9cb and pushed to 8.x. Thanks!