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.
When changing to a new shortcut set from the default shortcut set, the display breaks due to markup changing:
A default shortcut list item looks like this:
<li class="first leaf navbar-menu-item-processed navbar-level-1">
<div class="navbar-box">
<a href="/express2/node/add" class="navbar-menu-item">Add content</a>
</div>
</li>
After changing the shortcut set, a shortcut list item looks like this:
<li class="first leaf">
<a href="/express2/node/add">Add content</a>
</li>
Clearing the cache seems to fix the problem, but that should be added to the submit function rather than requiring the user to do it manually, especially if you have users that do not have that access.
Comment | File | Size | Author |
---|---|---|---|
#8 | navbar-shortcut-rebuild-theme--2653122-8.patch | 752 bytes | B-Prod |
#2 | navbar-shortcut_switch_menu_display-2653122-2-7.patch | 524 bytes | kevincrafts |
| |||
Screen Shot 2016-01-19 at 3.25.38 PM.jpg | 13.03 KB | kevincrafts | |
Screen Shot 2016-01-19 at 3.26.04 PM.jpg | 14.08 KB | kevincrafts |
Comments
Comment #2
kevincrafts CreditAttribution: kevincrafts commentedThis patch should be more targeted in the cache clearing, but it was the only way I was able to get the menu to display properly. Perhaps someone more familiar with this project could make this more targeted.
Comment #3
B-Prod CreditAttribution: B-Prod commentedAlmost same approach than #2, but:
Comment #5
B-Prod CreditAttribution: B-Prod commentedI clearly not understand the new testing system :-(
The tests result is 10:26:37 ERROR: No valid tests were specified.
I don't know what it means, but the patch clearly does not break other features...
Comment #8
B-Prod CreditAttribution: B-Prod commentedSame patch as #3, without testing.