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

Fix/menu list recreate mdcmenu on items change #2571

Merged
merged 4 commits into from
Oct 31, 2023

Conversation

TommyLindh2
Copy link
Contributor

Review:

  • Commits are atomic
  • Commits have the correct type for the changes made
  • Commits with breaking changes are marked as such

Browsers tested:

(Check any that applies, it's ok to leave boxes unchecked if testing something didn't seem relevant.)

Windows:

  • Chrome
  • Edge
  • Firefox

Linux:

  • Chrome
  • Firefox

macOS:

  • Chrome
  • Firefox
  • Safari

Mobile:

  • Chrome on Android
  • iOS

This because the mdcMenu.Items isn't in sync with
the rendered items thus prevent keyboard navigation
@TommyLindh2 TommyLindh2 requested a review from Kiarokh October 31, 2023 10:29
Copy link
Contributor

@Kiarokh Kiarokh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was already approved, but was merged to another branch, not in next. Now we merge it to next for real

@Kiarokh Kiarokh self-assigned this Oct 31, 2023
@Kiarokh Kiarokh added bug Something isn't working accessibility Issues that make it harder, or even impossible, for some specific group of users to use the software labels Oct 31, 2023
Copy link

Documentation has been published to https://lundalogik.github.io/lime-elements/versions/PR-2571/

@Kiarokh Kiarokh merged commit 1905ef9 into next Oct 31, 2023
11 checks passed
@Kiarokh Kiarokh deleted the fix/menu-list-recreate-mdcmenu-on-items-change branch October 31, 2023 13:54
@lime-opensource
Copy link
Collaborator

🎉 This PR is included in version 37.1.0-next.52 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lime-opensource
Copy link
Collaborator

🎉 This PR is included in version 37.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lime-opensource
Copy link
Collaborator

🎉 This PR is included in version 37.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Issues that make it harder, or even impossible, for some specific group of users to use the software bug Something isn't working released on @next released on @37.1.x released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants