Skip to content

#36 Basic stylesheet setup#37

Merged
yamenAl merged 4 commits into
mainfrom
recep-private
May 15, 2025
Merged

#36 Basic stylesheet setup#37
yamenAl merged 4 commits into
mainfrom
recep-private

Conversation

@Recss68

@Recss68 Recss68 commented May 14, 2025

Copy link
Copy Markdown
Owner

What does this change?

Images

image

How to review

  • Komen de kleuren overeen met het originele design van Drop & Heal
  • Is de typograpghy correct volgende het originele design
  • Graag aangeven als we eventuele aanpassingen kunnen doen

@Recss68
Recss68 requested a review from Mikiyas-hs May 14, 2025 12:02
@Recss68 Recss68 self-assigned this May 14, 2025
@Mikiyas-hs

Mikiyas-hs commented May 14, 2025

Copy link
Copy Markdown
Collaborator
  • De basis is goed opgesteld laten we ook spacing properties toevoegen zoals gap en padding

  • De headings en de P hebben allemaal ook een specifieke fonts voeg dit ook toe

@yamenAl

yamenAl commented May 15, 2025

Copy link
Copy Markdown
Collaborator

Bedankt voor het opzetten van de basis van de stylesheet volgens ons Figma-bestand.
Het ziet er al goed uit! Ik heb wel een aantal suggesties voor verbeteringen:

De hoofdkleuren komen niet volledig overeen met de originele stijl van Drop & Heal en de bijbehorende stylesheet. Je kunt mijn Huis-stijl bekijken van de vorige sprint om te zien hoe ik de kleuren daar heb geïmplementeerd als referentie.
https://github.com/yamenAl/user-experience-enhanced-website/blob/40acf9f53e2acd419108dbe9af67ffbe050c9920/public/assets/styles/huis-style.css#L13-L20

Typografie ziet er goed uit. Om de leesbaarheid op verschillende schermformaten te verbeteren, raad ik aan om responsive fontsizes toe te passen, zoals: font-size: clamp(1.5rem, 3vw, 2.5rem);

@Recss68

Recss68 commented May 15, 2025

Copy link
Copy Markdown
Owner Author

@yamenAl

Bedankt voor je feedback ik pas de kleuren van de rouwtaken aan in onze figma en zet ze in de stylesheet.

@Recss68
Recss68 requested a review from yamenAl May 15, 2025 17:06
@yamenAl
yamenAl merged commit 81a2ef0 into main May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants