fdnd-agency / deloitte

2 stars 1 forks source link

Docenten review #51

Closed koeenm closed 1 month ago

koeenm commented 1 month ago

Review van Justus

Logica van de svelte en het project

Punt 1

Styles kan gewoon in de static map. We hadden het nu in de routes, maar het is een statische file dus kan gewoon in de static map. En daarna in de head zetten van app.html. Alleen de global.css in de static map en hierin globale css zoals :root styling. En alle andere styles gewoon in de svelte files van de pagina's. De css voor een component in de bestand van de component zelf.

Punt 2

const url = ... in de .env zetten. (directus link)

Punt 3

error.svelte in de route map zetten en niet in aparte mappen van aparte pagina's.

Als je twee hele verschillende pagina's hebt dan wel verschillende error pagina's.

Punt 4

Hooks bekijken bij navigatie van website (advanced) voor het inloggen in de website.

Informatie over hooks in svelte:

https://kit.svelte.dev/docs/hooks

Hooks.server.js als bestand in de routes map

Punt 5

Om een beetje af te kijken hoe we components zouden kunnen gebruiken kunnen we kijken bij de fdnd components

https://github.com/fdnd/components/tree/main/src/lib

Dont repeat yourself

In een breakdownschets kunnen we ook kijken naar waar we componenten van zouden willen maken

Voor componenten met een atomic design te creeren:

https://bradfrost.com/blog/post/atomic-web-design/

Punt 6

Volgende sprint meer kijken naar svelte principes omarmen. En deze sprint nog focussen op de dingen die deze sprint nog te behalen zijn.