Katselmointi

[Jakousa]

Katselmointi

Sivuston käytettävyys

Mitä tein

• 16.8.2019 klo ~13:50

• Rekisteröidyin ja kirjauduin sisään

  • Lisäsin workoutin
  • Postailin viestejä
  • Katselin kuvia

Kokemus

• Ruudun leveys avasi sovelluksen mobiiliversiona. Pienempi leveys ok ja suurempi ok. Jokin ongelma ~1500px leveydellä.

• Hyvän näköinen softa

• Olisi hauska jos otat jonkin "draggable" kirjaston käyttöön.

• Uuden "workout" lisääminen kaipaisi paremman lomakkeen.

• Sivun päivittäminen ei toimi (muu kuin juurisivu)

• Toinen bookmarkkauksen este on faviconin jättäminen reactin omaksi

Koodi

• Kansiointi on tehty hyvin, kaikki löytyy juuri sieltä mistä odottaisi, controllerit, componentit etc.

• Todella hyvää ja luettavaa koodia.

• Ainoa pieni henkilökohtainen hiertäminen on jsx asettaminen sulkeiden riville:

  return ( <div>
    {screen.width < screen.height && props.feed.feed.map(post => 
      <Post key={post._id} post={post} />)}
    {props.feed.loading && <Spinner />}
  </div> )

vs

  return ( 
    <div>
      {screen.width < screen.height && props.feed.feed.map(post => 
        <Post key={post._id} post={post} />)}
      {props.feed.loading && <Spinner />}
    </div> 
  )

• Osan logiikasta voisi siirtää pois jsx / returnin sisältä ja siten parantaa luettavuutta: Katso esim https://github.com/miksa1987/Beast.Mode/blob/master/beast.mode/src/components/post/Comments.js ja lue https://reactjs.org/docs/conditional-rendering.html

Kokonaisuus

• Erinomainen projekti.

• On vaikea sanoa mitään pakollista kehitettävää. Mutta sivupäivitys on syytä korjata.

• Koodin laatu tukee jatkokehitystä, hienoa!

💪 / 🏋️‍♀️