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): mobile #1005

Merged
merged 1 commit into from
Jan 27, 2023
Merged

fix(menu): mobile #1005

merged 1 commit into from
Jan 27, 2023

Conversation

samuel-gomez
Copy link
Contributor

Related issue

Reference to the issue

fix #1004

Description of the issue

  • mobile menu not appear
  • fix some accessibilty issues

Person(s) for reviewing proposed changes

@korger-johan @arnaudforaison

Important

Before creating a pull request run unit tests

$ npm test

# watch for changes
$ npm test -- --watch

# For a specific file (e.g., in packages/context/__tests__/command.test.js)
$ npm test -- --watch packages/action

@samuel-gomez samuel-gomez merged commit 5306910 into master Jan 27, 2023
@delete-merged-branch delete-merged-branch bot deleted the feature/fix-mobile-menu branch January 27, 2023 12:49
@MartinWeb MartinWeb added the fix label Sep 13, 2023
@MartinWeb MartinWeb added this to the v2.1.0 milestone Sep 13, 2023
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 this pull request may close these issues.

[menu] not appear in mobile and some accessibility issue
4 participants