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

Page size limited to 500KB? #95

Closed
antarestupin opened this issue Oct 25, 2017 · 7 comments
Closed

Page size limited to 500KB? #95

antarestupin opened this issue Oct 25, 2017 · 7 comments

Comments

@antarestupin
Copy link
Contributor

antarestupin commented Oct 25, 2017

I don't see the arguments of limiting the page size to 500KB in the links under the recommandation in the performance section. I agree that pages should be light, but 500KB seems arbitrary.

As the ideal size of a page depends on too many things, I would recommend to replace this limit by the use of a performance budget, which is more flexible than a fixed limit and a good way to make the team responsible about the pages size. Tools like the performance budget calculator are very helpful to compute the budget.

Alternatively, if you want an interesting fixed limit, you can recommend to limit the size of the critical content to 14KB, so that the browser will be able to start rendering it during first TCP roundtrip. But that's an advanced practice, I'm not sure it would fit in this list.

@thedaviddias
Copy link
Owner

Hi @antarestupin, indeed I put that value at first based on some articles I read times ago. It's something that was already flagged in a previous ticket. I'm planning to review all these points and give more links after I launch the web app 😉.

Let's keep the issue open until we review it.

@thedaviddias
Copy link
Owner

@antarestupin Instead of page weight, maybe response-time is more relevant. We can for example, say "Your response-time need to be inferior to 10 seconds? What do you think? https://www.nngroup.com/articles/website-response-times/

@antarestupin
Copy link
Contributor Author

antarestupin commented Dec 8, 2017

@thedaviddias I agree, the initial goal should be expressed in term of seconds/ms as it's the actual user experience that you want to specify. But I don't think a strict value should be set in this checklist, because what's really important for the team is to define a goal, achieve it and keep it in consideration when changing things in the app. Then from this goal they can define an ideal page weight using tools like the one I mentioned above.

If you need inspiration for setting such a goal I recommend the RAIL model which defines ideal performance goals (for page load it's 1 second).

@antarestupin
Copy link
Contributor Author

antarestupin commented Jan 13, 2018

@thedaviddias I'm reading the latest version of Smashing Magazine's performance checklist and I see in the 7th section that they advise precise numbers concerning performance, to be exact:

  • a First Meaningful Paint under 1 second
  • a Time To Interactive under 5 seconds for the "average" configuration (a $200 Android on a slow 3G network with 400ms RTT and 400kbps transfer speed) and under 2 seconds for repeat visits
  • a critical file size under 170Kb gzipped

What do you think of using these numbers as a recommendation, or at least using the same kind of template?

@thedaviddias
Copy link
Owner

Absolutely @antarestupin, I thought about the same thing when I read the performance checklist. Feel free to push some modifications into the README :)

@antarestupin antarestupin mentioned this issue Jan 24, 2018
2 tasks
@stale
Copy link

stale bot commented Jul 7, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you have any question, please contact me directly at thedaviddias@gmail.com. Thank you for your contributions to that project!

@stale stale bot added the wontfix label Jul 7, 2018
@antarestupin
Copy link
Contributor Author

Oops sorry I forgot to close this issue.

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

No branches or pull requests

2 participants