navigation Logo does not communicated where the link is taking the user

Created on 17 April 2024, 8 months ago
Updated 16 May 2024, 7 months ago

Problem/Motivation

Steps to reproduce

navigation logo is not really communicated where the link is taking the user.
currently: output is an A tag with an image with alt text. No other string in link.

Proposed resolution

-use role="presentation" aria-hidden="true" for img/svg. (remove alt, aria-label)
-add hidden text in A tag. phrase

is a neutral phrase whether the user is currently on the active path or user is in deeper breadcrumb location.

Remaining tasks

User interface changes

none

API changes

none

Data model changes

none

🐛 Bug report
Status

Needs work

Version

11.0 🔥

Component
Navigation 

Last updated 1 day ago

No maintainer
Created by

🇨🇦Canada SKAUGHT

Live updates comments and jobs are added and updated live.
  • Needs usability review

    Used to alert the usability topic maintainer(s) that an issue significantly affects (or has the potential to affect) the usability of Drupal, and their signoff is needed. When adding this tag, make it easy to review the issue. Make sure the issue summary describes the problem and the proposed solution. Screenshots usually help a lot! To get sign-off on issues with the "Needs usability review" tag, post about them in the #ux channel on Drupal Slack, and/or attend a UX meeting to demo the patch and get direct feedback from designers/UX folks/product management on next steps. If an issue represents a significant new feature, UI change, or change to the general "user experience" of Drupal, use Needs product manager review instead. See the scope of responsibilities for product managers.

Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @SKAUGHT
  • 🇨🇦Canada SKAUGHT

    UX/A11Y question?
    - should we change the img to decorative (and remove alt).
    - svg will be using 'aria-label' -- can we swap simply this to be 'hidden'?

  • Status changed to Needs review 8 months ago
  • Pipeline finished with Success
    8 months ago
    Total: 573s
    #149379
  • Pipeline finished with Success
    8 months ago
    Total: 623s
    #149380
  • Pipeline finished with Success
    8 months ago
    #149399
  • 🇩🇪Germany rkoller Nürnberg, Germany

    about the question should the logo be decorative or not. probably making the icon decorative might be the right step. www.tpgi.com and www.deque.com for example are using the alt attribute while the www.a11yproject.com is using the aria-hidden=true attribute for the logo. in case of tpgi and deque, those are both companies and they try to communicate that their company branding is in place here. but in the case of the navigation module knowing that there is an icon doesnt provide any extra benefit, to do it properly you would even have to describe it, that you are seeing a blueish rectangular button with rounded edges with a white drupal drop logo (since in this form it isnt used anywhere else). so making it decorative and focus on the link's purpose i might consider in the context of a navigation menu the better choice.

    in regards of the label. going with just Homepage i am not sure if that is that clear. In the old admin toolbar you had Back to site. That made it clear that you are going somewhere entirely else. With homepage one might question are you going to /admin or what does Homepage refer to?
    Adding a detail that you are going to the landing page of your default theme in the front end might clarify things. So i wonder maybe go with something like Frontpage default theme or Homepage default theme. That would include for one that you go to the frontpage/homepage and that it is the frontpage/homepage of the default theme. but "default theme" could also be too abstract in case the user is unaware of the concept of the default theme in drupal. as an alternative you could use the installed theme's name instead, but that could also throw off the user. Or even return to the label that was used in the admin toolbar and go with Back to site or completely different option that was not raised yet?

  • 🇨🇦Canada SKAUGHT

    IMO: 'default theme' info is too much info.. i understand the thought.

    we have talked about toolbar and 'back to site' which itself has additional redirect logic (which others also thought is actually confusing as the landing point then is in consistant..).

    'back to site': means nothing if you are already on the homepage, thus (as an english phrase) 'homepage' feels like it will work without having to dynamically change the title on . of course, hoping to avoid 'frontpage' (:
    -> should we switch the title when on the path? maybe this is the problem.

  • Pipeline finished with Success
    8 months ago
    Total: 225s
    #150051
  • Status changed to Needs work 8 months ago
  • 🇪🇸Spain ckrina Barcelona

    Moving to Needs work after getting Navigation into core.

  • 🇨🇦Canada SKAUGHT

    SKAUGHT changed the visibility of the branch 3441586-navigation-logo-does to hidden.

  • 🇨🇦Canada SKAUGHT

    SKAUGHT changed the visibility of the branch 3441586-navigation-logo-does to hidden.

  • 🇨🇦Canada SKAUGHT

    SKAUGHT changed the visibility of the branch 3441586-navigation-logo-does to active.

  • 🇪🇸Spain ckrina Barcelona
  • First commit to issue fork.
  • Status changed to Needs review 8 months ago
  • 🇮🇳India gauravvvv Delhi, India
  • Status changed to Needs work 7 months ago
  • 🇺🇸United States smustgrave

    Think the MR needs to be opened.

    Also feels like something we can add a test assertion somewhere to an existing test, if one exists know the tests are still WIP.

  • Pipeline finished with Failed
    7 months ago
    Total: 676s
    #172110
  • 🇮🇳India mithun s Bangalore

    Mithun S made their first commit to this issue’s fork.

  • Pipeline finished with Failed
    7 months ago
    Total: 618s
    #174293
  • Pipeline finished with Success
    4 months ago
    Total: 421s
    #273709
  • Pipeline finished with Success
    4 months ago
    Total: 5893s
    #273751
  • Pipeline finished with Skipped
    4 months ago
    #274724
  • Pipeline finished with Success
    4 months ago
    Total: 1256s
    #274706
  • Pipeline finished with Skipped
    4 months ago
    #274792
  • Pipeline finished with Success
    3 months ago
    Total: 50s
    #290374
  • Pipeline finished with Success
    3 months ago
    Total: 53s
    #290385
  • Pipeline finished with Success
    about 2 months ago
    Total: 404s
    #329845
  • Pipeline finished with Failed
    about 2 months ago
    Total: 278s
    #329854
  • Pipeline finished with Failed
    about 2 months ago
    Total: 299s
    #329859
  • Pipeline finished with Success
    about 2 months ago
    Total: 375s
    #329869
Production build 0.71.5 2024