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.
If a block menu is used in the navigation bar, bullets have been removed and links are inline, but the block titles and margin still exists. It would be useful if these were removed.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1144240-remove_block_title_adjust_menu_block_margin_in_navbar.patch | 469 bytes | KrisBulman |
Comments
Comment #1
KrisBulman CreditAttribution: KrisBulman commentedthis patch applies the above changes in STARTERKIT only
Comment #2
KrisBulman CreditAttribution: KrisBulman commentedComment #3
KrisBulman CreditAttribution: KrisBulman commentedJust noticed I should have done this in the sass navigation file.. When attempting to approach it, I sadly failed on getting the ruby gem sass installed on my Debian vm. :(
Comment #4
barraponto CreditAttribution: barraponto commentedKris, you should try using rvm https://rvm.beginrescueend.com/ it helps a lot. Takes some effort to get used, but you can try all gem combinations you can think of (I use it to move from sass/compass betas and stables, and testing out my drupal compass plugin).
Comment #5
barraponto CreditAttribution: barraponto commentedbtw, shouldn't we look for a proper accessibility solution for this? display: none hides things from screen readers as well.
Comment #6
KrisBulman CreditAttribution: KrisBulman commentedIf the idea is to give the title access to screen readers, then it would work to replace display:none with this
unless there is a better way by adding an element-hidden class to menus that appear in the navigation region.. might be overkill though.
Comment #7
mgiffordActually, better to just use the pre-defined class element-invisible on any item you don't want to appear on the browser, but do want to be visible to assistive technology. See:
http://drupal.org/update/themes/6/7#element-hidden
Comment #9
JohnAlbinZen now has a element-invisible() mixin in the _custom.scss partial.
Comment #10
JohnAlbinComment #11
JohnAlbinThis makes sense.
Comment #12
JohnAlbinLooks like stripping the .content div from the block.tpl broke this functionality in the interim.
Comment #13
JohnAlbinFixed. http://drupalcode.org/project/zen.git/commitdiff/7591874
Thanks, Kris!