search

RenzoWille / look-and-feel-styleguide

Ontwerp en maak met een team een styleguide om te gebruiken voor een opdracht van een opdrachtgever.
https://renzowille.github.io/look-and-feel-styleguide/
MIT License
0 stars 0 forks source link

Code Review Checklist #9

Open DivaniNL opened 2 hours ago

DivaniNL commented 2 hours ago

Indeling van de stylesheet

Er zijn geen comments geplaatst om onderscheid te maken tussen onderdelen.

Consistente naamgeving

Gebruik in css zo min mogelijk id selectors -> https://github.com/RenzoWille/look-and-feel-styleguide/blob/main/index.css#L48

Naamgeving en begrijpelijkheid

Dit is goed.

Volledigheid van de stylesheet

Deze stylesheet bevat geen states.

Custom properties op juiste element

Headings hebben een class van het heading level. Bijvoorbeeld: https://github.com/RenzoWille/look-and-feel-styleguide/blob/main/index.html#L69-L74. Je kan de css ook plaatsen op h1{Code...}

Herhaling van code

Over het algemeen is dit goed. Alleen bij de buttons worden regels css herhaald die eigenlijk voor alle buttons gelden. Zie: https://github.com/RenzoWille/look-and-feel-styleguide/blob/main/index.css#L170-L177

Feedback gegeven door: @karima002 @fatimahilali @mikiyas-hs @naddybs @raffa

KyanTG commented 2 hours ago

@KyanTG @dvdalfsen @senneb31