- 🇮🇳India Harish1688 India
Hi,
Tried to replicate the issue on local with same Drupal 9.4.8 with admin toolbar 3.2.1. and testing the things steps wise as provided but not facing any issue on hover state when switch to vertical to horizontal toolbar mode. Hover working fine and by access by tabs focus also working fine.can you please provide more info or (UI) to issue, so would be able to generate.
- 🇮🇳India Harish1688 India
Hi @thalemn,
Tried with same info as you provided to reproduced the issue on local. but not able to reproduce the issue on local, it's working fine.
attached a GIf for reference. - 🇺🇸United States thalemn
@Harish1688
Thanks for providing the gif.
There appears to be a step missing in your gif demo: when you switch from horizontal to vertical, click on an admin link to open the page (for example, click on Block layout link to open the Block layout page (/admin/structure/block). After that page opens (with the admin toolbar in the vertical position), switch the toolbar back to horizontal. Then the horizontal links do not hover for me.
Can you please try again?
- Status changed to Needs review
over 1 year ago 8:17am 8 August 2023 - last update
over 1 year ago Patch Failed to Apply - 🇮🇳India swatidhurandhar
Hi, I was able to reproduce the issue with above steps. Please find the attached patch for the fixes.
Thanks - 🇧🇹Bhutan Pema Namgyel
Thanks buddy for the patch. I have reviewed the patch and its fixing the issue. Attaching the video for the same.
- 🇺🇸United States thalemn
Thank you. I have applied the patch in #9 and the hover issue has been fixed.
- last update
over 1 year ago Patch Failed to Apply - Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.4 & MySQL 8last update
11 months ago Waiting for branch to pass - 🇮🇳India keshavv India
I can reproduce the issue.
Set the menubar vertical.
Reload the page.
Change the menubar position to horizontal and try to hover on the menu without reloading the screen.
Hover will not work.However, The given patch failed to apply to me on version 3.4 so I have created a new for it.
Please review.
Thank you. - 🇮🇳India Sandeep_k New Delhi
@keshavv, Tested the shared Patch 3320479-14.patch on Drupal version- 9.5.11. The patch was applied successfully and looks good to me.
Testing Steps:- Drupal 9.5> Add the Admin Toolbar module.
- Switch to the vertical toolbar.
- Click on the link in the vertical toolbar (any module link for example).
- Switch back to the horizontal toolbar.
- Hover over horizontal toolbar links to reproduce this issue.
- Download the shared patch and Apply.
- Reverify this issue by following the above steps.
Testing Results:
After applying the patch, the hover is working fine while we switch back to the horizontal toolbar from the vertical toolbar. RTBC++ - 🇮🇳India Kanchan Bhogade
Hi,
I've tested the Patch 3320479-14.patch on Drupal version- 9.5.12.
The patch was applied successfully...Followed the same steps as mentioned in #15
Testing Results:
The hover functionality is working fine when the user switches back to the horizontal toolbar from the vertical toolbar.
RTBC++attaching videos
Keeping in "needs review" for code verification