Problem/Motivation

The admin toolbar dropdown icon (indicated in the screenshot) on the content page does not appear to function as expected. Users are unable to access additional options or features when clicking the icon.

Observed Behavior
No menu or options are displayed when clicking the dropdown icon. The button appears to be non-functional.

Relevant Screenshots
Gif showing the hamburguer before

Steps to reproduce

  1. Log in as a user with administrative privileges.
  2. Navigate to the Content page (/admin/content).
  3. Zoom in your browser (300% should be enough).
  4. Observe the dropdown menu icon in the upper right corner of the page (highlighted in the screenshot).
  5. Attempt to click or interact with the icon.
  6. Expected Behavior
  7. Clicking the dropdown menu icon should reveal additional options or functionality, allowing administrators to access relevant features directly from the content page.

Proposed resolution

Added icon to close menu and js to proper apply css based on the state of menu open/closed

Remaining tasks

Code review & test.

User interface changes

It changes hamburger menu behaviour under tabs component.

Introduced terminology

NA

API changes

NA

Data model changes

NA

Release notes snippet

Added close icon for hamburger menu.

Issue fork drupal-3500425

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

haris khan jadoon created an issue. See original summary.

haris khan jadoon’s picture

Issue summary: View changes
StatusFileSize
new43.7 KB
haris khan jadoon’s picture

StatusFileSize
new186.21 KB
cilefen’s picture

Component: other » Olivero theme
Issue tags: +Needs steps to reproduce

The menu appears to be working in the screenshot you posted. Additionally, I can't reproduce the problem on Drupal 10.4 with the steps to reproduce as written above.

haris khan jadoon’s picture

StatusFileSize
new1.63 KB

Yes, it's working, but when it's opened, the icon or appearance doesn't change as it should.

Like this

cilefen’s picture

Status: Active » Postponed (maintainer needs more info)

The issue summary says that "users are unable to access additional options or features when clicking the icon", but in fact the problem is that the icon should change when used. Do I understand now?

Which theme is this?

haris khan jadoon’s picture

Its Administration theme (claro)

haris khan jadoon’s picture

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

The issue is on the administration theme

quietone’s picture

Version: 10.4.x-dev » 11.x-dev
Component: Olivero theme » Claro theme
Issue summary: View changes
Status: Needs work » Postponed (maintainer needs more info)

Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to the Core change policies.

Restored the standard template, which we use to track progress and others tasks on an issue.

I tested this on Drupal 11.x, standard install, following the step in the issue summary. At step 3, there is no "dropdown menu icon in the upper right corner of the page". Is something missing? Is there custom code on the site?

julio_retkwa made their first commit to this issue’s fork.

julio_retkwa’s picture

I've added a possible fix for that, would be helpful to add on test instructions that if you are on vanilla drupal site you can just zoom in the browser under step 3
If anyone could review, that would be great :)

julio_retkwa’s picture

Status: Postponed (maintainer needs more info) » Needs review
smustgrave’s picture

Status: Needs review » Needs work
Issue tags: +Needs tests

Probably be good to get test coverage added for this. Twig template will probably need a CR as contrib themes would have to update theirs.

julio_retkwa’s picture

#14 Hi @smustgrave sorry I'm not sure if I get it right. I should add some test coverage on which stage exactly? (I presume nightwatch?) and the CR would stand for Change Request on contrib themes yes?

julio_retkwa’s picture

Status: Needs work » Needs review
bnjmnm’s picture

Category: Bug report » Feature request
Issue tags: +Needs issue summary update

This isn't a bug - it's a feature request.

The issue summary is still inaccurate. It should be presented as a desire to have a different icon present depending on open/closed state.

smustgrave’s picture

Status: Needs review » Needs work

So my fault I actually should of mentioned the issue summary is incomplete. Proposed solution should almost always be filled in.

Maybe nightwatch (I'm not great with those) or a functional javascript test maybe? So we can confirm that the functionality is still working to make sure the bug doesn't comeback.

And correct CR is change record, you'll see the button on the right at the top for adding one. Can be real simple too

julio_retkwa’s picture

Issue summary: View changes
bnjmnm’s picture

Issue summary: View changes
julio_retkwa’s picture

Issue summary: View changes
StatusFileSize
new331.52 KB
julio_retkwa’s picture

Status: Needs work » Needs review
snehal-chibde’s picture

StatusFileSize
new52.57 KB
new51.79 KB

hello, I have checked this on Drupal 11.x-dev, as per the steps mentioned. The MR is working as expected.
On click of hamburger, when the menu is open the hamburger icon changes to close icon. Added before and after screenshots for reference.

sandip’s picture

Hi @julio_retkwa, I’ve reviewed your MR — the changes are looking good overall. One suggestion I’d like to propose is to enhance accessibility by adding the aria-hidden attribute to the <span> elements wrapping the hamburger and close icons.

In addition, we should update the JavaScript to toggle the aria-hidden values based on the menu state. This ensures that only the currently visible icon is announced by screen readers, aligning with accessibility best practices.

Let me know what you think!

julio_retkwa’s picture

Hi @sandip thank for the suggestion! - I've added that on my last commit

sandip’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new202.06 KB

Changes looks good to me attaching a Video for better understanding. Also corrected the branch versions in CR. Moving this issue to RTBC.

needs-review-queue-bot’s picture

Status: Reviewed & tested by the community » Needs work
StatusFileSize
new91 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

julio_retkwa’s picture

Status: Needs work » Reviewed & tested by the community
julio_retkwa’s picture

Can anyone review this?

nod_’s picture

Title: Admin Toolbar Dropdown Icon Missing Functionality on Content Page » Add a close icon for Claro primary tabs on mobile
needs-review-queue-bot’s picture

Status: Reviewed & tested by the community » Needs work
StatusFileSize
new91 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

vasantha deepika’s picture

Assigned: Unassigned » vasantha deepika

I'll look into this.

sandip’s picture

Assigned: vasantha deepika » Unassigned

As there is no response from @vasantha for one week so i am picking it up.

sandip’s picture

Status: Needs work » Reviewed & tested by the community

I am again moving it to RTBC as the MR is clean after the rebase. Here is the After image: After Image Link that i already posted in #26.

One minor thing is if i am not wrong the CR should be updated for version 11.3 right?

  • nod_ committed 4f4ff211 on 11.x
    Issue #3500425 by cilefen, julio_retkwa, smustgrave, bnjmnm, sandip,...

nod_’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs steps to reproduce, -Needs tests, -Needs issue summary update

I don't think this warrant a change record, i won't publish it.

Committed 4f4ff21 and pushed to 11.x. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

quietone’s picture

Per #37 I am deleting the change record. It was simply one sentence saying a feature was added which is not something for a change record.

Status: Fixed » Closed (fixed)

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