Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Onhover not working on some meganav dropdown items [tablet/mobile view] #14319

Open
petesfrench opened this issue Sep 20, 2024 · 0 comments · May be fixed by #14332
Open

Onhover not working on some meganav dropdown items [tablet/mobile view] #14319

petesfrench opened this issue Sep 20, 2024 · 0 comments · May be fixed by #14332
Labels

Comments

@petesfrench
Copy link
Contributor

Summary

The onhover effect not working on some meganav items. Specifically links with the class p-navigation__dropdown-item.

Potential fixes:

  1. Update the pattern to match that of Vanilla (this might have unexpected effects on functionality)
<li class="">
  <a href="/desktop" class="p-navigation__dropdown-item 
      p-link--inverted" onclick="event.stopPropagation()" tabindex="0">
    <span>Ubuntu Desktop</span>
    <br>
    <small class="is-muted">Fast, modern and secure Linux</small>
  </a>
</li>
  1. Apply the onhover effect here

Steps to reproduce the behavior

  1. Go to https://ubuntu.com/ on tablet or mobile screen size
  2. Open the meganav by clicking the 'menu'
  3. Navigation to the bottom layer of any navigation item. ex. 'Products' > 'Featured'.
  4. Hover your mouse over the dropdown items
  5. See that no onhover effect is applied

Expected behavior

Onhover the background changes to #313131.

Current:
image
Expected:
image

You can see this in action in this vanilla example

Browser/device details

Ubuntu, Chrome

Reported from

GitHub

@codewithnick codewithnick linked a pull request Sep 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant