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

Chore: Improve initial page load time #128

Open
6 tasks
nelsonic opened this issue Aug 13, 2022 · 5 comments
Open
6 tasks

Chore: Improve initial page load time #128

nelsonic opened this issue Aug 13, 2022 · 5 comments
Assignees
Labels
chore a tedious but necessary task often paying technical debt discuss Share your constructive thoughts on how to make progress with this issue enhancement New feature or enhancement of existing functionality help wanted If you can help make progress with this issue, please comment! priority-2 Second highest priority, should be worked on as soon as the Priority-1 issues are finished question A question needs to be answered before progress can be made on this issue starter A beginner-friendly issue that is a good starting point for a new person T4h Time Estimate 4 Hours technical A technical issue that requires understanding of the code, infrastructure or dependencies

Comments

@nelsonic
Copy link
Member

At present, PageSpeed Insights tells us that the performance of the MVP is "100"
https://pagespeed.web.dev/report?url=https%3A%2F%2Fmvp.fly.dev%2F ... πŸ’― πŸ€·β€β™‚οΈ
image

1.9 s Time to Interactive 🐌

Apparently an App that takes 1.9 s Time to Interactive scores a 100 on PageSpeed.
Maybe compared to some ridiculously slow-to-load news site this is good.
But just to render a list of items? it seems incredibly slow to me.
What can we do to lower this to ~400ms on a 3G connection?
Can we pre-render and cache the page and differ connecting to the LiveView socket? πŸ’­

Todo

Remember the purpose of the MVP is Learning.
Both our own learning of how to build Apps using PETAL πŸ‘©β€πŸ’»
and testing our hypothesis that people want to use a UX-focussed tool to maximise their personal effectiveness. ❀️ βœ…
In our quest to test that hypothesis, we need to minimise the page load time ⏳
So that the person viewing the MVP for the first time is pleasantly surprised 😍
and we know we've done everything we can to Never Waste Anyone's TimeTM πŸ‘Œ

@nelsonic nelsonic added enhancement New feature or enhancement of existing functionality help wanted If you can help make progress with this issue, please comment! question A question needs to be answered before progress can be made on this issue technical A technical issue that requires understanding of the code, infrastructure or dependencies T4h Time Estimate 4 Hours priority-2 Second highest priority, should be worked on as soon as the Priority-1 issues are finished starter A beginner-friendly issue that is a good starting point for a new person discuss Share your constructive thoughts on how to make progress with this issue chore a tedious but necessary task often paying technical debt labels Aug 13, 2022
@nelsonic
Copy link
Member Author

@SimonLab want to pick this up and write-up a perf.md with your quest? πŸ’­
If we could shave off any time from the "Time to Interactive" 🐌 it would be amaze! 🀞

@nelsonic
Copy link
Member Author

Latest result: https://pagespeed.web.dev/report?url=https%3A%2F%2Fmvp.fly.dev%2F
image

The speed index has increased significantly. πŸ€·β€β™‚οΈ
I think we need to re-think the rendering of the home page and cache it all for people who have never interacted with the app before. πŸ’­

@SimonLab
Copy link
Member

Current metrics on my side
image

@nelsonic
Copy link
Member Author

Yeah, similar on my side:

image

this is "acceptable" for a web app. πŸ’­
It would be awesome if we could integrate this check into our CI/CD pipeline.

@nelsonic
Copy link
Member Author

Accessibility and "Best Practices" have both declined to 79 and 83 respectively:
https://pagespeed.web.dev/report?url=https%3A%2F%2Fmvp.fly.dev%2F
image

Looks like we can tidy up the markup:
image

Who wants to pick this up on Monday? πŸ™

@nelsonic nelsonic moved this to πŸ”– Ready for Development in dwyl app kanban Nov 25, 2022
@nelsonic nelsonic moved this from πŸ”– Ready for Development to πŸ“‹ Backlog in dwyl app kanban Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore a tedious but necessary task often paying technical debt discuss Share your constructive thoughts on how to make progress with this issue enhancement New feature or enhancement of existing functionality help wanted If you can help make progress with this issue, please comment! priority-2 Second highest priority, should be worked on as soon as the Priority-1 issues are finished question A question needs to be answered before progress can be made on this issue starter A beginner-friendly issue that is a good starting point for a new person T4h Time Estimate 4 Hours technical A technical issue that requires understanding of the code, infrastructure or dependencies
Projects
Status: πŸ“‹ Backlog
Development

No branches or pull requests

2 participants