mokhtarakle / grote-themas-sprint-19

Release candidate v2 voor project grote themas

Mokhtar

Hi there 👋

My name is Mokhtar. I am an 23 year old frontend developer living in Amsterdam.

Stats

Does not include forked repositories

Skills

💻 Frontend Developer

grote-themas-sprint-19's Issues

Controleer het contrast voor alle pictogrammen.

Naamgeving consistentie

Onderstaand gebruik je een Nederlandse naamgeving terwijl de andere folders in het Engels zijn.

https://github.com/MokhtarAkle/grote-themas-sprint-19/blob/main/src/lib/organisms/WerkvormCard.svelte

Zorg ervoor dat alle animaties voldoen aan de prefers-reduced-motionmediaquery.

Feedback CMD student question

I'm Rose, a Communication and Multimedia Design third year student. My teacher Sanne 't Hooft at the Amsterdam University of Applied Sciences redirected me to the study that Mokhtar follows, because my team and I were running into a problem with SvelteKit that nobody on our side could fix. Mokhtar helped me, and without him, we would not have been able to fix the problem.

The problem had to do with a complex function within the Mapbox-GL library, where a function was not updated when it's paramater value changes. We spent a lot of time on this, but he knew exactly what to look for and was able to quickly determine the next steps as we worked through the problem.. and the ones that revealed themselves, as is inevitable when you fix one thing.. but he handled it!

He is incredibly patient, knows what to look for, and working with him went very smoothly. Thanks a lot for the help and good luck in the future! :)

Voor afbeeldingen die tekst bevatten, zorg ervoor dat de alt-beschrijving de tekst van de afbeelding bevat.

Geef een unieke waarde op title voor elke pagina of weergave.

Controleer of de focusvolgorde van het toetsenbord overeenkomt met de visuele lay-out.

Responsiveness verbeteren kleine schermen

https://github.com/MokhtarAkle/grote-themas-sprint-19/wiki/Responsiveness-test

Testen op performance

Code design review

Tips

Misschien per rij 4 kaarten vertonen inplaats van 3 want nu heb je daar te veel witruimte.

De kleur van de icon is slecht zichtbaar vanwege de kleur geel.

De witruimte is niet consistent begrijp is een minimal website maar door vaste margin en padding te geven en dat alles mooi uitgelijnd is zal de website minder leeg lijken.

Misschien hier een achtergrond kleur erbij doen zodat het minder leeg lijkt.

De login centreren en misschien beetje vergroten.

Tops

• Heel mooi hoe je de login systeem heb gemaakt
• Je hebt clean consistent code

Verplaats foto's van assets naar CMS

Zorg ervoor dat er een zichtbare focusstijl is voor interactieve elementen waarnaar wordt genavigeerd via toetsenbordinvoer.

Detailpagina heeft dit nog niet

Kopelementen moeten in een logische volgorde worden geschreven.

Zorg ervoor dat links herkenbaar zijn als links.

Project live zetten

Het huidige project live zetten onder mijn controle

Zorg voor een lineaire contentstroom.

Design review

Zorg ervoor dat alle imgelementen een attribuut hebben alt.

Testen op responsiveness

Project installeren

Release candidate branch lokaal werkend krijgen i.c.m .env en CMS.

<h1> en <h2> lijkt niet erg van elkaar te verschillen

Ik zie dat de h1 in de homepagine even groot is als de h2 in overzichts section. Misschien kan je de h1 ietsjes groter maken om het meer opvallender maken

Nieuwe werkvormen toevoegen

Serveer foto's in webp/avif waar mogelijk

Bevestig dat transcripties beschikbaar zijn voor audio

Zorg ervoor dat formulierinvoerfouten na indiening in de lijst boven het formulier worden weergegeven.

Sla kopniveaus niet over.

Feedback op Code-review

De opdracht voor de tweedejaars was om een code review te doen met een eerstejaars op het gebruik van EJS en partials. Mokthar heeft toen een code review voor mij gedaan en alles in een Github issue geschreven : Geschreven Issue door Mokthar. Tijdens de code review vroeg hij mij ook een aantal vragen over bepaalde EJS termen en wat bepaalde regels code deden. Op een paar vragen had ik geen antwoord en probeerde hij het duidelijk uit te leggen waardoor ik het gelijk beter begreep.

Daarnaast heb ik ook een code review gekregen voor mijn HTML & CSS wat eigenlijk niet bij de opdracht hoorde maar wel fijn en handig was omdat een tweedejaars natuurlijk anders kijkt naar code als een eerstejaars.

Testen op A11y

Voeg lazy loading toe waar mogelijk

Detailpagina werkvormen - Toegankelijkheid

Toegankelijkheid verbeteren van de detailpagina

Controleer het contrast van de randen van invoerelementen (tekstinvoer, keuzerondjes, selectievakjes, enz.).

Bevestig de aanwezigheid van bijschriften in videos

Weekplanning maken

Expliciete groottes toevoegen foto's

Zorg ervoor dat fout-, waarschuwings- en successtatussen niet visueel worden gecommuniceerd door alleen maar kleur.

Make sure that button, a, and label element content is unique and descriptive.

Controleer uw inhoud in gespecialiseerde browsermodi.

Zorg ervoor dat de inhoud van het button, a, en labelelement uniek en beschrijvend is.

Responsiveness verbeteren grote schermen

https://github.com/MokhtarAkle/grote-themas-sprint-19/wiki/Responsiveness-test