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

Fix icons in 1.9.8 #1680

Merged
merged 1 commit into from Apr 15, 2024
Merged

Fix icons in 1.9.8 #1680

merged 1 commit into from Apr 15, 2024

Conversation

emma-sg
Copy link
Member

@emma-sg emma-sg commented Apr 15, 2024

Closes #1676.

Loads sl-icon synchronously to get correct base path when running webpack-dev-server.

Partially cherry-picked from 548261e by @SuaYoo.

Confirmed icons load with local dev & with a local prod build where they didn't before these changes.

Loads `sl-icon` synchronously to get correct base path when running
webpack-dev-server.
@emma-sg emma-sg added bug Something isn't working front end Requires front end dev work labels Apr 15, 2024
@emma-sg emma-sg requested review from ikreymer and SuaYoo April 15, 2024 20:29
@emma-sg emma-sg self-assigned this Apr 15, 2024
@emma-sg emma-sg linked an issue Apr 15, 2024 that may be closed by this pull request
@ikreymer ikreymer merged commit f76b83e into 1.9.8-release Apr 15, 2024
2 checks passed
@ikreymer ikreymer deleted the 1.9.8-fix-icons branch April 15, 2024 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working front end Requires front end dev work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Icons are missing throughout the app (v1.9.7)
2 participants