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

[bug] Icons not rendering on prod #441

Closed
theyokohamalife opened this issue Apr 1, 2024 · 6 comments · Fixed by #468
Closed

[bug] Icons not rendering on prod #441

theyokohamalife opened this issue Apr 1, 2024 · 6 comments · Fixed by #468
Assignees
Labels
bug Something isn't working

Comments

@theyokohamalife
Copy link
Contributor

theyokohamalife commented Apr 1, 2024

Icons are currently not rendering on prod for the site logo, healthcare professionals, and languages.

Image

@theyokohamalife theyokohamalife added the bug Something isn't working label Apr 1, 2024
@theyokohamalife
Copy link
Contributor Author

We'll tackle this on Wednesday in a group coding session.

@NabbeunNabi
Copy link
Contributor

I was wondering @ermish and @theyokohamalife if you have looked at this and think it is a good add to the project, vue-svg-loader?

Pros

  1. Imports SVG as components so looks clean in the code.
  2. Has very few dependencies. Only 2

Cons

  1. Has dependencies so it does add more to the project

@ermish
Copy link
Contributor

ermish commented May 14, 2024

I was wondering @ermish and @theyokohamalife if you have looked at this and think it is a good add to the project, vue-svg-loader?

Pros

  1. Imports SVG as components so looks clean in the code.
  2. Has very few dependencies. Only 2

Cons

  1. Has dependencies so it does add more to the project

a best practice is to have all images (including svg's) to be cacheable which means they should not be inlined. Unfortunately vue-svg-loader inlines them, so it's not a valid option

@NabbeunNabi
Copy link
Contributor

But as mentioned yesterday it seems like the way we did it on the about us page fits the requirements for svgs and makes it cacheable so we should proceed that way?

@ermish
Copy link
Contributor

ermish commented May 14, 2024

But as mentioned yesterday it seems like the way we did it on the about us page fits the requirements for svgs and makes it cacheable so we should proceed that way?

yeah, that should be fine. Just documenting the thoughts for anyone else reading this. 👍

@ermish
Copy link
Contributor

ermish commented May 14, 2024

I'll have a PR ready for this soon 🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants