Skip to content

Navigation Menu

Explore
By company size
By use case
By industry
View all solutions
Topics
- AI
- DevOps
- Security
- Software Development
- View all
Explore
- GitHub Sponsors
  Fund open source developers
- The ReadME Project
  GitHub community articles
Repositories
- Enterprise platform
  AI-powered developer platform
Available add-ons
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

vaadin / web-components Public

Notifications You must be signed in to change notification settings
Fork 85
Star 493

Code
Issues 1k
Pull requests 9
Actions
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Security
Insights

[menu-bar] Add possibility to disable roving tabindex#7525

Merged

web-padawan merged 17 commits intovaadin:mainvaadin/web-components:mainfrom jcgueriaud1:feat/menu-bar-navjcgueriaud1/web-components:feat/menu-bar-navCopy head branch name to clipboard

Aug 12, 2024

+216-7

Conversation Commits17 (17)Checks Files changed

Commits

Commits on Aug 12, 2024

6718 [menu-bar] Add possibility to disable roving tabindex

jcgueriaud1
authored and
web-padawan
committed
Revert the change in the demo

jcgueriaud1
authored and
web-padawan
committed
Update the code to fix the navigation between submenus. Test to be done

jcgueriaud1
authored and
web-padawan
committed
Test the navigation with tabs

jcgueriaud1
authored and
web-padawan
committed
Remove a useless import

jcgueriaud1
authored and
web-padawan
committed
Copy the helpers from app-layout to menu-bar

jcgueriaud1
authored and
web-padawan
committed
Add the this._expandedButton !== null as a condition

jcgueriaud1
authored and
web-padawan
committed
Apply suggestions from code review
web-padawan
committed
Update packages/menu-bar/src/vaadin-menu-bar-mixin.js
web-padawan
committed
Update packages/menu-bar/src/vaadin-menu-bar-mixin.js
web-padawan
committed
refactor: simplify menu-bar mixin code
web-padawan
committed
test: add missing newline
web-padawan
committed
test: split tests for switching submenu on Tab and Shift Tab
web-padawan
committed
test: inline sendKeys usage and remove helpers file
web-padawan
committed
refactor: simplify if condition to remove nested block
web-padawan
committed
docs: add comments to clarify switching submenu
web-padawan
committed
test: add tabNavigation to typings test
web-padawan
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Docs
Contact

You can’t perform that action at this time.