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

Inconsistency in MenuData.icon between Flow and Hilla views without icon #20114

Closed
Legioth opened this issue Oct 2, 2024 · 0 comments · Fixed by #20137
Closed

Inconsistency in MenuData.icon between Flow and Hilla views without icon #20114

Legioth opened this issue Oct 2, 2024 · 0 comments · Fixed by #20137

Comments

@Legioth
Copy link
Member

Legioth commented Oct 2, 2024

Description of the bug

For a Flow view without an icon (i.e. just a plain @Menu), MenuData.icon contains "" whereas a Hilla view without icon (i.e. export const config = { menu: {} }) has null.

Expected behavior

Expected that the same value is used in both cases so that application logic that renders the menu doesn't have to check for both cases.

Minimal reproducible example

24.5 pre starter from Start

Versions

  • Vaadin / Flow version: 24.5.0.beta4
@mshabarov mshabarov moved this to 🔖 High Priority (P1) in Vaadin Flow bugs & maintenance (Vaadin 10+) Oct 2, 2024
@tltv tltv self-assigned this Oct 3, 2024
tltv added a commit that referenced this issue Oct 3, 2024
tepi pushed a commit that referenced this issue Oct 4, 2024
@github-project-automation github-project-automation bot moved this from 🔖 High Priority (P1) to ✅ Closed in Vaadin Flow bugs & maintenance (Vaadin 10+) Oct 4, 2024
@github-project-automation github-project-automation bot moved this from 🟢Ready to Go to Done in Vaadin Flow ongoing work (Vaadin 10+) Oct 4, 2024
vaadin-bot added a commit that referenced this issue Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

3 participants