Problem/Motivation

  1. While reviewing with a small group, people were now sure what the 'Extent Sidebar' is now extending while desktop screen sizes expand/contracted during general run through. Title lacks connection to the navigation tool, users did not re-associate objects.
  2. Without javascript in 'small breakpoints' we see the 'expand sidebar' button. Without JS this does nothing. Navigation does not let user get into any Drupal admin path.

Steps to reproduce

using Drupal 11/10.3 standard profile.
enable Navigation module
with your browser, disable Javascript. make window less than 500px.
notice 'a bar at top' and bottom 'Expand sidebar'. this does nothing.

Proposed resolution

Also include 'navigation logo' in this area and breakpoint. As it serves are landmark for users and link to get into 'administration' when nojs and in this breakpoint.
- reminder: navigation has an template/svg.logo.twig -- Expand around with the custom logo use (with parent/wrapping link) then return that into the layout/navigation.html.twig so that it can be reused here.

Remaining tasks

choices:
- should this link to go /admin/index. (currently goes LOGO goes to Frontpage.
- should the logo link to 'return to site' type functionality.
ie 'admin/index'->front->user_setting->return-to-side/(last known page?)
- should we add a setting for user to set their choice. default/none will be (front or admin/index).

User interface changes

enhance JS degrade of Navigation 'bar at top/mobile trigger'

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
navigation-mobile-js-degrade.png189.15 KBskaught

Comments

SKAUGHT created an issue. See original summary.

skaught’s picture

Title: navigations 'expand sidebar' width javascipt offers no administration access. » navigations 'expand sidebar' without javascript offers no administration access.
skaught’s picture

Issue summary: View changes
skaught’s picture

Title: navigations 'expand sidebar' without javascript offers no administration access. » navigations 'expand sidebar' without javascript offers no links to admin pages.
skaught’s picture

Issue summary: View changes
skaught’s picture

Issue summary: View changes
skaught’s picture

Title: navigations 'expand sidebar' without javascript offers no links to admin pages. » navigations 'expand sidebar' (title) lacks connection to flow
skaught’s picture

Issue summary: View changes
skaught’s picture

Issue summary: View changes
ckrina’s picture

Not sure what this issue is for: to add a logo when the Navigation is collapsed on mobile or to make it work on mobile without JS? If it's the first, the designs have already explored the logo on mobile and discarded it. The reason is that it will need to hold more elements beyond the existing toggle. If it's to get this working without JS, #3396174: The toolbar should be usable without JS. Let's clarify what this is for and make sure that issues focus on solving 1 problem only. :)

anruether’s picture

Status: Active » Postponed (maintainer needs more info)
xmacinfo’s picture

I agree, Navigation “expand/collapse” on desktop and “show/hide” on small breakpoints, like on a mobile phone.

The “Expand sidebar” label is too generic, and is not clearly linked to the new Navigation toolbar. The proper context is not clear.

It would be more accurate to write “Display admin menu”. That way, there is no need to add a logo, which would not make the label more understandable.

I prefer “Display admin menu”, but “Show admin menu” or “Expand admin menu” also work.

And to those who prefer short and sweet, we could use:

“Admin menu” or simply “Admin”.

“Admin” alone is perfectly fine next to the hamburger menu icon.

xmacinfo’s picture

Status: Postponed (maintainer needs more info) » Needs work

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.

mherchel’s picture

Had Zoom with @mgifford, @rkoller, and @kat-shaw.

This is a bit confusing of an issue. Per @ckrina in #10, we're not adding the logo. This issue might be to relabel the button.

Can we get clarification?

mgifford’s picture

This is not a stable blocker in our current understanding of the problem.