sumbitpage layout rework + frontend testen

[masinnae]

frontend testen voor volgende files zijn gemaakt

• FilesInput.vue
• ProjectMiniCard.vue
• SubmitCard.vue
• SubmitForm.vue
• SubmitView.vue

[reyniersbram]

Bij het uitvoeren van de testen krijg ik nog deze warning. Ook met de upgraded versie van @vue/test-utils die nu beschikbaar is op de dev-branch.

stderr | tests/components/project/submit/SubmitCard.spec.ts > SubmitCard > render if not error
[Vue warn]: injection "Symbol(router)" not found. 
  at <SubmitForm projectId=1 > 
  at <VCol> 
  at <VRow> 
  at <VContainer key=1 class="card-container" > 
  at <VCard variant="flat" > 
  at <SubmitCard projectId=1 ref="VTU_COMPONENT" > 
  at <VTUROOT>

[reyniersbram]

Een opmerking over de afbeelding hieronder. Dit geeft mij het gevoel dat de 'Add Files' titel ook slaat op het inputveld voor opmerkingen. Ik denk dat dit beter zou zijn als de indentation van het inputveld en de 'Submit' knop tegenover die titel weggehaald zou worden. Een andere optie zou kunnen zijn om die titel weg te halen, aangezien dit in de tekst van de knop herhaald wordt.

[masinnae]

Een opmerking over de afbeelding hieronder. Dit geeft mij het gevoel dat de 'Add Files' titel ook slaat op het inputveld voor opmerkingen. Ik denk dat dit beter zou zijn als de indentation van het inputveld en de 'Submit' knop tegenover die titel weggehaald zou worden. Een andere optie zou kunnen zijn om die titel weg te halen, aangezien dit in de tekst van de knop herhaald wordt.

Ja ik ga denk ik eens de hele layout herzien want nu zit die in een container met rijen en kolommen terwijl dit wat overbodig is aangezien alles gwn onder mekaar staat. De indent zou dan ook weg moeten zijn.

[reyniersbram]

Ja ik ga denk ik eens de hele layout herzien

Doe je dit nog in deze PR? Zo niet keur ik deze al goed want ziet er voor de rest wel goed uit.

[masinnae]

Doe je dit nog in deze PR? Zo niet keur ik deze al goed want ziet er voor de rest wel goed uit.

in deze pr samen met een fix voor die error voor de test

[masinnae]

• testen opgekuist zodat niet elke test de router mock of authentication mock nodig heeft
• layout aangepast (geen container meer met indentation)
• "Voeg bestanden toe"-titel aangepast naar "Bestanden"

[pieterjanin]

top, kga meteen mergen