Amberhva/the-valley-nike-case-study

Issues

• bewijslast verwoorden

  #22 opened 2 years ago by r20222
  0

• class names

  #18 opened 2 years ago by r20222
  0

• navigation

  #20 opened 2 years ago by r20222
  0

• De nav bar tijdens scrollen

  #13 opened 2 years ago by DaanCarst
  0

• Logo op desktop

  #16 opened 2 years ago by RickVellingaa
  0

• width:100vw?

  #19 opened 2 years ago by r20222
  0

• Font size in de root

  #10 opened 2 years ago by DaanCarst
  0

• Weet je zeker dat je hier een div nodig hebt? Wellicht een anchor gebruiken (die linkt naar de beginpagina)?

  #5 opened 2 years ago by joostf
  0

• images mobile-first

  #21 opened 2 years ago by r20222
  0

• Goed dat je content er allemaal in staat

  #17 opened 2 years ago by r20222
  0

• Probeer met meer content en html naar de code en design review te komen?

  #11 opened 2 years ago by joostf
  0

• mooie nav

  #14 opened 2 years ago by r20222
  0

• voor de <p> text is de text "weight" aan de dikke kant deze zou ik dunner maken

  #12 opened 2 years ago by D-spacezero
  0

• Sticky nav

  #8 opened 2 years ago by RickVellingaa
  0

• Goed gebruik comments

  #4 opened 2 years ago by Tristandemuijnck
  0

• HTML is erg overzichtelijk door de notes die overal bij staan :)

  #6 opened 2 years ago by DaanCarst
  0

• Heel goed beschrijving bij code

  #7 opened 2 years ago by r20222
  0

• Comments in code

  #3 opened 2 years ago by RickVellingaa
  0

• Custom properties

  #2 opened 2 years ago by r20222
  0

• Website taal instellen, staat nu nog op engels

  #1 opened 2 years ago by D-spacezero
  0

• Favicon

  #15 opened 2 years ago by Tristandemuijnck
  0

• Kan je dit ook zonder jQuery maken?

  #9 opened 2 years ago by joostf
  1