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

Menu Bar Text visible through Usage & Styling Header #2355

Closed
mrgreywater opened this issue Apr 6, 2023 · 6 comments
Closed

Menu Bar Text visible through Usage & Styling Header #2355

mrgreywater opened this issue Apr 6, 2023 · 6 comments

Comments

@mrgreywater
Copy link

mrgreywater commented Apr 6, 2023

I just noticed the MenuBar Button text is visible through the new Usage & Styling Header.

chrome_HukMZQ9LGZ.mp4
@jouni
Copy link
Member

jouni commented Apr 6, 2023

I’m not able to reproduce this on macOS Safari, Chrome, or Firefox. What OS and browser are you using?

@mrgreywater
Copy link
Author

mrgreywater commented Apr 6, 2023

This page: https://vaadin.com/docs/latest/components/menu-bar
With Chrome Version 112.0.5615.50 (Official Build) (64-bit)
The same happens with Firefox 111.0.1 (64-bit)
OS: Windows 11 Enterprise 10.0.22621

All extensions disabled. It only seems to happen to the java components, not when you select the typescript components.

image

edit: Bug also exists on my android phone with chrome and firefox.

@jouni
Copy link
Member

jouni commented Apr 11, 2023

It only seems to happen to the java components, not when you select the typescript components.

Thanks, that was the missing piece of information. I can reproduce it now as well 👍

@jouni
Copy link
Member

jouni commented Apr 11, 2023

The issue comes from these styles: https://github.com/vaadin/web-components/blob/main/packages/menu-bar/src/vaadin-menu-bar-button.js#L20-L23

When using Flow/Java, the additional <vaadin-menu-bar-item> element gets inserted. The TypeScript version just has a plain text node inside the <vaadin-menu-bar-button> element.

I have no memory why these styles are there in the first place (originally for slotted vaadin-context-menu-item elements). @web-padawan, or @tomivirkki, do you remember?

@jouni
Copy link
Member

jouni commented Apr 11, 2023

Oh, there we go: vaadin/vaadin-menu-bar#69

I wonder if this fix is still needed?

@jouni
Copy link
Member

jouni commented Apr 11, 2023

I pushed a fix/workaround to Design System Publisher (the documentation website builder), which should solve this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants