Add Collapsed Status for Menu Block Title

Created on 25 March 2024, 9 months ago
Updated 23 April 2024, 8 months ago

Problem/Motivation

When the navigation panel is collapsed, there's no indication as to which menu is which. This can be confusing for users to navigate.

  • Navigate to admin/config/user-interface/navigation-block
  • Edit/configure an existing navigation block
  • Check the checkbox to enable the title for that menu and save
  • That block will now have a title on the left toolbar.
  • Collapse the toolbar: a dot will take the place of the title

Proposed resolution

When the navigation is collapsed, replace the text title with a dot icon, remove any hover interactions that match link hover effects, and instead have the menu title show in a tooltip (see Figma designs).

๐Ÿ“Œ Task
Status

Fixed

Version

1.0

Component

Code

Created by

๐Ÿ‡บ๐Ÿ‡ธUnited States KeyboardCowboy Denver, CO, USA

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

Merge Requests

Comments & Activities

  • Issue created by @KeyboardCowboy
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States KeyboardCowboy Denver, CO, USA

    We need to get the design mocks for this one from Figma.

  • ๐Ÿ‡ช๐Ÿ‡ธSpain ckrina Barcelona
  • Assigned to kostyashupenko
  • ๐Ÿ‡ท๐Ÿ‡บRussia kostyashupenko Omsk

    Gonna try it

  • Issue was unassigned.
  • Status changed to Postponed 9 months ago
  • ๐Ÿ‡ท๐Ÿ‡บRussia kostyashupenko Omsk

    Postponed because this functionality is already partially done here https://www.drupal.org/project/navigation/issues/3427659 ๐Ÿ“Œ Implement drawer functionality Needs review

  • Status changed to Active 9 months ago
  • ๐Ÿ‡ช๐Ÿ‡ธSpain ckrina Barcelona

    This has bee implemented in ๐Ÿ“Œ Implement drawer functionality Needs review and ๐Ÿ“Œ Implement the new designs to the drawer Needs review . But one small detail is missing: the pointer style should be defined so the browser doesn't show a "?".

  • Pipeline finished with Success
    9 months ago
    Total: 266s
    #139505
  • Status changed to Needs review 9 months ago
  • ๐Ÿ‡ช๐Ÿ‡ธSpain ckrina Barcelona

    Added styles based on the final designs, plus updated the issue summary to reflect the final designs and testing steps.

  • ๐Ÿ‡ฉ๐Ÿ‡ชGermany rkoller Nรผrnberg, Germany

    One problem to note, if you try to tab through the collapsed sidebar, menu block titles are not focusable. with tabbing that structuring element is unavailable to screenreader users. the only cue screenreader users have is that there are four headers by default (in case the display of title is active):

    shortcuts
    content
    administration
    rkoller

    just based on the announcement it is not clear that those are actual section headers, you would need the visual context. in addition tabbing is the only way to get all the menu block titles as well as menu items in consecutive order. in the voice over rotor the components are scattered, the block titles are headers, top level items with a sub menu are buttons and top level items without a submenu are links.

  • ๐Ÿ‡ช๐Ÿ‡ธSpain ckrina Barcelona

    @rkoller thanks for the feedback! Could you open a follow-up issue for that and add it into the accessibility plan? It's not that I don't care about accessibility, but we need to move and iterate fast to get the navigation to a better point, and the classic core treatment of issues where nothing gets in until it's perfect will slow us down in this very key moment. :)

  • Pipeline finished with Success
    9 months ago
    Total: 222s
    #141670
  • Pipeline finished with Success
    9 months ago
    Total: 226s
    #141694
  • ๐Ÿ‡ช๐Ÿ‡ธSpain ckrina Barcelona

    @rkoller opened ๐Ÿ“Œ Improve how and what navigation block titles are communicating to screenreader users Active as a follow-up for the keyboard issue.

  • Status changed to RTBC 9 months ago
  • ๐Ÿ‡ฉ๐Ÿ‡ชGermany rkoller Nรผrnberg, Germany

    There are only two concerns i have. For the first i've already opened up a follow up issue in the context of keyboard and screenreader accessibility (will add another comment about the needs for sighted keyboard users there as well).
    the other detail is about the dot, but that concern shouldnt be a blocker for this issue. overall it is difficult to communicate with just an icon that the user should hover over the icon (when i first saw the dot i thought some icon was missing - like for a missing unicode glyph) . creating another follow up issue wouldnt make much sense since that issue wouldnt be actionable and with the navigation issue queue moving over into the core issue cue soon it would be cruft for the time being. @ckrina made a note about the dot and will continue ideating on it over with the folks in the design team. therefore no followup issue necessary in this case. therefore this issue could be set to rtbc

  • Pipeline finished with Skipped
    9 months ago
    #141920
    • ckrina โ†’ committed b18e344f on 1.x
      Issue #3435989 by ckrina, rkoller: Add Collapsed Status for Menu Block...
  • Status changed to Fixed 9 months ago
  • ๐Ÿ‡ช๐Ÿ‡ธSpain ckrina Barcelona

    Thanks @rkoller! Merging this for now and we can make the improvements in a follow-up.

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

  • Pipeline finished with Success
    6 months ago
    Total: 52s
    #216470
  • Pipeline finished with Success
    4 months ago
    Total: 289s
    #269867
  • Pipeline finished with Skipped
    2 months ago
    #316874
  • Pipeline finished with Success
    about 2 months ago
    Total: 156s
    #328873
  • Pipeline finished with Success
    about 2 months ago
    Total: 179s
    #328876
  • Pipeline finished with Canceled
    14 days ago
    Total: 91s
    #366815
  • Pipeline finished with Success
    14 days ago
    Total: 147s
    #366817
  • Pipeline finished with Canceled
    14 days ago
    Total: 72s
    #366819
  • Pipeline finished with Success
    14 days ago
    Total: 146s
    #366822
Production build 0.71.5 2024