-
Notifications
You must be signed in to change notification settings - Fork 1
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
Peer feedback Jaymar #62
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Readme
De Readme is nog niet veranderd van de standaard svelte installatie Readme. in principe kun je die houden maar de andere punten die ik mis zijn:
Performance, Accessibility & Responsiveness
Naast de Titel en alt attributen aanpassen is de toegankelijkheid erg goed met een score van 94.
De performance van de site is niet optimaal. De grootste boosdoeners van de 67 score zijn de images. Ze zijn niet de juiste size en kunnen ook in een ander format (webp/avif) wat ook helpt met performance. Daarnaast duurt de LCP ook erg lang. Hier zou even naar gekeken moeten worden want er wordt aangegeven dat het komt door een statische blockquote. De TBT is aangegeven als langer dan 50ms in sommige gevallen wat ook een reden is van een slechtere performance.
Tijdens de browser test heb ik de volgende browsers gebruikt:
Op andere browsers was de TBT een stuk beter, dus het ligt waarschijnlijk aan google chrome.
De pagina is niet helemaal responsive. het werkt wel en is bruikbaar, maar is niet helemaal geoptimaliseerd voor telefoon.
De pagina is niet responsive. Het design is duidelijk niet gemaakt voor mobiel dus deze pagina moet in tegenstelling tot de overview pagina gerefactored worden om responsive te zijn.
Voor de performance moet de LCP (de login image) sneller later. De image moet net zoals op de andere pagina in next-gen formats en geresized worden naar de actuele grootte
De pagina is ook niet responsive. Het design is alleen gemaakt voor desktop dus deze pagina moet ook gerefactored worden om responsive te zijn.
De LCP klopt mogelijk niet, maar word aangegeven als een reden dat de site trager laadt. De reden dat ik dat denk is omdat het wordt aangegeven dat de h2 de LCP is (maar de content wordt wel ingeladen vanuit de database, dus er moet even naar gekeken worden voor de zekerheid)
The text was updated successfully, but these errors were encountered: