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.
Kickstart profile duplicate user menu instead of adding the link to current default user menu.
Links should be created in Drupal default user menu and kickstart theme stylesheet class need to point to the correct system block.
Find and Replace
.block-menu-user-menu
with
.block-system-user-menu
Comment | File | Size | Author |
---|---|---|---|
#5 | kickstart_profile-1774632-5.patch | 898 bytes | mglaman |
#3 | duplicate-user-menus.png | 23.48 KB | rootwork |
Comments
Comment #0.0
deep-silver CreditAttribution: deep-silver commentedadded code
Comment #1
zoon_unit CreditAttribution: zoon_unit commentedI too, wonder why there are duplicate user menus in Commerce Kickstart. It is highly confusing when moving menu blocks around. The two user menu blocks are named the same. (although their machine names differ) This also affects styling, since one menu uses CSS classes of "block-user-menu" while the other uses "block-menu-user-menu."
This seemingly unnecessary duplicate is wasteful and confusing.
Comment #2
lonehorseend CreditAttribution: lonehorseend commentedI'm trying to set the active trail for the "my account" menu item whenever someone is on the user path and this duplicate menu issue seems to be interfering.Never mind. Fixed my issue. But I still agree with everyone else that duplication of the user menu is confusing.
Comment #2.0
lonehorseend CreditAttribution: lonehorseend commentedcode
Comment #3
rootworkJust noticed this as well and it seems odd it hasn't been addressed. Ideally, I agree that the CK menu items should be added to the existing menu, not create a new one. But if it is going to create a new one, let's give it a different name, like "Commerce User Menu" or "Commerce Kickstart User Menu" to reduce confusion.
For reference, the "real" user menu (machine name: user-menu) contains two links: My account, and Log out. The CK-created user menu (machine name: menu-user-menu) contains four links: Log in, Create account, My account [disabled], and Log out.
Comment #4
lsolesen CreditAttribution: lsolesen commented@rootwork Do you want to take a stab on fixing this?
Comment #5
mglamanPatch to change admin display name to "Kickstart User menu" for clarity.
Build tests: https://travis-ci.org/commerceguys/commerce_kickstart/builds/72192235
Comment #7
mglaman