Feature: users #7

Merged

jonasclaes merged 10 commits from te-f-users into master

2021-11-12 10:23:38 +00:00

jonasclaes commented

2021-11-09 21:12:00 +00:00

(Migrated from github.com)

Eerste review:
Werkpunten:

User index:
- Te veel kleuren, het is een business applicatie, kleurenschema is vooral zwart, wit, grijs, als we knoppen hebben, blauw gebruiken, delete knop rood, edit knop lichtblauw
- Omvormen naar lijst stijl met minimale informatie zodat mobile experience fijner is (zie Files overzicht)
- Bewerk, verwijder knop verhuizen naar individuele user pagina
- De pagination is goed! Engiste puntje, maak de achtergrond van de actieve pagina nummer wat donkerder zodat je dat beter kan zien.
- De zoekbox zou ik meer stylen richting het Files gedeelte, je kan in principe die hele header overnemen
- De create user knop zou ik ook meer stylen zoals een normale button (ook weer, zie Files voor een voorbeeldje)
User edit:
- Deze pagina vind ik goed gemaakt, alleen staat het titeltje "Edit The User Information" een beetje niet op zijn plaats, consider moving this under the Register title
- Is het een idee om met een grid van Tailwind te werken om bv First name en Last name naast elkaar te zetten?
- Ik mis nog enkele velden, namelijk: Telefoonnummer, actief ja/nee (checkbox)
- Ook weer teveel kleur. Liever de form wit houden, en met shadows werken, zo krijg je toch dat "card" gevoel
Alert:
- De alert vind ik goed gemaakt!
- De readability moet je wel eens nakijken, ik zou met witte tekst op de blauwe achtergrond werken. Als je niet zeker weet of t aan een standaard voldoet: https://webaim.org/resources/contrastchecker/ is een goede checker.
- Ik mis nog een kruisje om de notificatie weg te klikken
- Kan je de notificatie als overlay maken? Zodat hij bovenop alle content verschijnt ipv naar beneden drukt?
User show (individuele):
- Enkele tips/ideeen
- Laat zien wanneer deze het laatst is bewerkt en aangemaakt
- Laat alle properties zien zoals Firstname, lastname, username, email, phonenumber, actief status, de rechten zijn voor later
- Voeg de knoppen toe van edit en delete in de correcte kleuren bovenaan in de header eventueel
Algemeen:
- In de views folder, geen mappen met hoofdletters maken, op Linux (en uiteindelijk de webhost) geeft dat problemen. Filenames van componenten met hoofdletter, dat was ook zo, dus goed gedaan. Paginas zonder hoofdletter, gewoon alles lowercase.
- Over het algemeen zit de structuur ook goed, je moet enkel nog wat wijzigen onder resources/views/users, daar staan nog wat lege redundante bestanden.
- Ik heb enkele kleine wijzigingen gepushed, vergeet niet te pullen morgenochtend

Nog niet mergen

Eerste review: Werkpunten: - User index: - Te veel kleuren, het is een business applicatie, kleurenschema is vooral zwart, wit, grijs, als we knoppen hebben, blauw gebruiken, delete knop rood, edit knop lichtblauw - Omvormen naar lijst stijl met minimale informatie zodat mobile experience fijner is (zie Files overzicht) - Bewerk, verwijder knop verhuizen naar individuele user pagina - De pagination is goed! Engiste puntje, maak de achtergrond van de actieve pagina nummer wat donkerder zodat je dat beter kan zien. - De zoekbox zou ik meer stylen richting het Files gedeelte, je kan in principe die hele header overnemen - De create user knop zou ik ook meer stylen zoals een normale button (ook weer, zie Files voor een voorbeeldje) - User edit: - Deze pagina vind ik goed gemaakt, alleen staat het titeltje "Edit The User Information" een beetje niet op zijn plaats, consider moving this under the Register title - Is het een idee om met een grid van Tailwind te werken om bv First name en Last name naast elkaar te zetten? - Ik mis nog enkele velden, namelijk: Telefoonnummer, actief ja/nee (checkbox) - Ook weer teveel kleur. Liever de form wit houden, en met shadows werken, zo krijg je toch dat "card" gevoel - Alert: - De alert vind ik goed gemaakt! - De readability moet je wel eens nakijken, ik zou met witte tekst op de blauwe achtergrond werken. Als je niet zeker weet of t aan een standaard voldoet: https://webaim.org/resources/contrastchecker/ is een goede checker. - Ik mis nog een kruisje om de notificatie weg te klikken - Kan je de notificatie als overlay maken? Zodat hij bovenop alle content verschijnt ipv naar beneden drukt? - User show (individuele): - Enkele tips/ideeen - Laat zien wanneer deze het laatst is bewerkt en aangemaakt - Laat alle properties zien zoals Firstname, lastname, username, email, phonenumber, actief status, de rechten zijn voor later - Voeg de knoppen toe van edit en delete in de correcte kleuren bovenaan in de header eventueel - Algemeen: - In de views folder, geen mappen met hoofdletters maken, op Linux (en uiteindelijk de webhost) geeft dat problemen. Filenames van componenten met hoofdletter, dat was ook zo, dus goed gedaan. Paginas zonder hoofdletter, gewoon alles lowercase. - Over het algemeen zit de structuur ook goed, je moet enkel nog wat wijzigen onder resources/views/users, daar staan nog wat lege redundante bestanden. - Ik heb enkele kleine wijzigingen gepushed, vergeet niet te pullen morgenochtend ### Nog niet mergen