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

(i18n): Potential Site Performance Improvements #41213

Closed
ShaunSHamilton opened this issue Feb 21, 2021 · 3 comments
Closed

(i18n): Potential Site Performance Improvements #41213

ShaunSHamilton opened this issue Feb 21, 2021 · 3 comments
Labels
platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. status: discussing Under discussion threads. Closed as stale after 60 days of inactivity.

Comments

@ShaunSHamilton
Copy link
Member

As your average dev, I spend a lot of my time with my devtools open. As your un-average dev, I read way too much minified JS, which is why I stumbled upon the freeCodeCamp sourcemap.

I notice it ships with all languages included. Perhaps this does not create as large a performance impact as I think, but once we add more languages, surely it might?

I assume linking to the file like this does not work: https://www.freecodecamp.org/app-67cba5e849cb47817635.js
So, it is just the largest .js file downloaded when on freecodecamp.org/ that is not preset-env. It is about 552kB (over network).

@ShaunSHamilton ShaunSHamilton added the status: discussing Under discussion threads. Closed as stale after 60 days of inactivity. label Feb 21, 2021
@raisedadead raisedadead added the platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. label Feb 22, 2021
@ojeytonwilliams
Copy link
Contributor

I could be wrong, but I don't believe sourcemaps get loaded until you open the console. If someone could confirm that's the case, I think they're worth keeping.

@RandellDawson
Copy link
Member

@ojeytonwilliams I believe you are correct.

@ShaunSHamilton
Copy link
Member Author

Ah. Yes, I see... well, then ignore me 😄
Thank you, both, for pointing that out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. status: discussing Under discussion threads. Closed as stale after 60 days of inactivity.
Projects
None yet
Development

No branches or pull requests

4 participants