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
CommentFileSizeAuthor
#5 kickstart_profile-1774632-5.patch898 bytesmglaman
#3 duplicate-user-menus.png23.48 KBrootwork

Comments

deep-silver’s picture

deep-silver commented
Issue summary: View changes

added code

zoon_unit’s picture

zoon_unit commented

I 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.

lonehorseend’s picture

lonehorseend commented

I'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.

lonehorseend’s picture

lonehorseend commented
Issue summary: View changes

code

rootwork’s picture

rootwork
he/him
Primary language English
Location 🇺🇸 US-Pacific 🌎 Chinook (Multnomah, Clackamas) & Cowlitz lands 🌹 Portland, OR
commented
Title: Kickstart profile duplicate user menu. » Kickstart profile creates a duplicate user menu
Version: 7.x-2.0-beta2 » 7.x-2.10
Issue summary: View changes
StatusFileSize
new duplicate-user-menus.png23.48 KB

Just 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.

Screenshot of duplicate user menus

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.

lsolesen’s picture

lsolesen commented
Version: 7.x-2.10 » 7.x-2.14
Category: Bug report » Feature request
Status: Active » Postponed (maintainer needs more info)

@rootwork Do you want to take a stab on fixing this?

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new kickstart_profile-1774632-5.patch898 bytes

Patch to change admin display name to "Kickstart User menu" for clarity.

Build tests: https://travis-ci.org/commerceguys/commerce_kickstart/builds/72192235

  • mglaman committed 2dd991f on 7.x-2.x 
    Issue #1774632: Kickstart profile creates a duplicate user menu
  • mglaman committed 6203c7b on 7.x-2.x
    Merge pull request #141 from mglaman/1774632-namespace-user-menu

Issue...
mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.