Unchecking the content preview in navigation layout will change the navigation icon to text.

Created on 27 February 2025, about 2 months ago

Problem/Motivation

Unchecking the "Show Content Preview" checkbox in the navigation layout will change the navigation icons to text.

Steps to reproduce

1). Install latest version of drupal11.
2). Install Navigation module from extend
3). go to Configuration > UserInterface > Navigation Layout and try to uncheck the content preview check box.

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

šŸ› Bug report
Status

Active

Version

11.1 šŸ”„

Component

navigation.module

Created by

šŸ‡®šŸ‡³India rinku jacob 13 Kerala

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @rinku jacob 13
  • Merge request !11329Make navigation links are clickable ā†’ (Open) created by rinku jacob 13
  • šŸ‡®šŸ‡³India rinku jacob 13 Kerala

    The above MR will fix the navigation link click issue. However, for the second-level menu, if there is a submenu, the page immediately refreshes. I need to work on fixing that issue. For the navigation link fix, Iā€™m awaiting feedback from the maintainers to confirm if Iā€™m on the right track. Once I receive approval, I will focus on addressing the second issue I discovered. Adding screen records for the reference.

  • Pipeline finished with Failed
    about 2 months ago
    Total: 786s
    #436718
  • šŸ‡®šŸ‡³India rinku jacob 13 Kerala

    Even though I have made the menus clickable, they are not directing to the corresponding pages. Need to work on fixing this as well.

  • First commit to issue fork.
  • Pipeline finished with Failed
    21 days ago
    Total: 181s
    #459005
  • Pipeline finished with Failed
    21 days ago
    Total: 130s
    #459110
  • Pipeline finished with Failed
    21 days ago
    Total: 6762s
    #459124
  • Pipeline finished with Failed
    17 days ago
    Total: 99s
    #462203
  • Pipeline finished with Failed
    17 days ago
    Total: 4241s
    #462210
  • Pipeline finished with Success
    17 days ago
    Total: 492s
    #462291
  • šŸ‡ŖšŸ‡øSpain plopesc Valladolid

    Added new approach to try to address this issue.

    When the user lands in the Navigation blocks page, there is a button to enable the edition mode. Only when the edition mode is enabled, LB taker over the navigation.

    Once changes are saved or discarded, the user is back to normal mode, where the left bar navigation is fully functional.

  • šŸ‡ŖšŸ‡øSpain plopesc Valladolid
  • šŸ‡ØšŸ‡¦Canada m4olivei Grimsby, ON

    Looks great! Really like this approach. Just some super nit picks for naming. Otherwise looking good. I would favor this approach over messing with the styles.

  • Pipeline finished with Failed
    9 days ago
    #469191
  • Pipeline finished with Success
    8 days ago
    #469199
  • šŸ‡ŖšŸ‡øSpain plopesc Valladolid

    Thank you for your review @m4olivei. Applied your suggestions.

    Back to Needs Review.

  • šŸ‡®šŸ‡³India sagarmohite0031

    Hello,
    Tested and verified on Drupal 11.
    MR applied successfully.
    But issue is still there, When I'm clicking on on any option its redirect me to Edit layout for Navigation layout Add to Default shortcuts

    Steps to reproduce
    1). Install latest version of Drupal 11.
    2). Install Navigation module from extend
    3). go to Configuration > User Interface > Navigation Layout and try to click on any Navigation links.

    Please check Attachment-

  • šŸ‡ØšŸ‡¦Canada m4olivei Grimsby, ON

    Looks great! Nice improvement. Thanks @plopesc!

  • Pipeline finished with Success
    6 days ago
    Total: 361s
    #471458
  • šŸ‡ŖšŸ‡øSpain ckrina Barcelona

    +1 to the RTBC on a UX perspective while testing this locally, after being asked by @catch to +1 the UX changes.

    It would be great though that UI changes that have considerable UX changes are reviewed by a UX person before RTBC it. And to make UX people life easier it would be great that any RTBC comes with a proper issue summary update and screenshots or videos to help other UX reviewers without git knowledge to see and understand the changes.

    That said, 2 things:
    1. Great work everyone with this change! It takes away on of the worst UX behaviors from this page.
    2. We really need to start a plan to remove Layout Builder from this UI. It wasn't designed for this and it's an overkill that results in a very poor UX.

Production build 0.71.5 2024