Update menu-bar to use vaadin-menu-bar-item
component for sub-menus
#3010
Labels
enhancement
New feature or request
refactor
Internal improvement
requires new major
This would be a breaking change
theme
v24
To be done in V24
Motivation
Right now we have a weird case where
vaadin-menu-bar
usesvaadin-context-menu-item
:web-components/packages/menu-bar/src/vaadin-menu-bar-buttons-mixin.js
Lines 268 to 273 in b4a9cea
This caused some confusion among the team members, not to mention the actual users.
Proposed solution
vaadin-menu-bar-item
component that would extendvaadin-context-menu-item
.web-components/packages/menu-bar/theme/lumo/vaadin-menu-bar-button-styles.js
Lines 16 to 36 in b4a9cea
Note
Similar fix was done in Vaadin 22 for
vaadin-time-picker-item
extendingvaadin-combo-box-item
.The text was updated successfully, but these errors were encountered: