KEMBAR78
chore: fix icon dev page by jouni · Pull Request #10159 · vaadin/web-components · GitHub
Skip to content

Conversation

@jouni
Copy link
Member

@jouni jouni commented Sep 12, 2025

Replace the JS module import with the stylesheet.

@jouni jouni requested a review from vursen September 12, 2025 11:24
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Icon</title>
<script type="module" src="./common.js"></script>
<link rel="stylesheet" href="../packages/vaadin-lumo-styles/src/props/icons.css">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dev server should automatically inject lumo.css, which already includes icons.css, so this link might be unnecessary.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lumo CSS gets removed when running yarn start by default and only loaded with yarn start:lumo which breaks icons. I can modify the config to not remove all CSS files, then we can keep this.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, let’s at least remove the JS import which no longer exists :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pushed a change to not remove .css files from dev pages when loaded explicitly, like in case of icon.html - this should also allow to debug aura.html page when running yarn start without yarn start:aura.

@sonarqubecloud
Copy link

@web-padawan web-padawan merged commit a3183e4 into main Sep 12, 2025
9 checks passed
@web-padawan web-padawan deleted the chore/icon-dev-page branch September 12, 2025 12:30
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 25.0.0-alpha10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants