Skip to content

CSS voor de footer + afbeelding van de website toegevoegd#45

Merged
Recss68 merged 3 commits into
mainfrom
yamen-private
May 15, 2025
Merged

CSS voor de footer + afbeelding van de website toegevoegd#45
Recss68 merged 3 commits into
mainfrom
yamen-private

Conversation

@yamenAl
Copy link
Copy Markdown
Collaborator

@yamenAl yamenAl commented May 15, 2025

Deze pull request bevat:

  1. Styling (CSS) voor de footer.
  2. Toevoeging van benodigde afbeeldingen en iconen voor gebruik op de website.

yamenAl added 2 commits May 15, 2025 19:32
Adding the CSS for the footer in the header CSS file
photos and i coins for the website
@yamenAl yamenAl requested review from Mikiyas-hs and Recss68 May 15, 2025 18:21
@Recss68
Copy link
Copy Markdown
Owner

Recss68 commented May 15, 2025

@yamenAl
Ziet er goed uit, is het niet handig als we de rouwtaak foto's de namen geven van de rouwtaak? Of had je daar een ander idee voor? Laat me weten dan merge ik het

image

@yamenAl
Copy link
Copy Markdown
Collaborator Author

yamenAl commented May 15, 2025

@yamenAl Ziet er goed uit, is het niet handig als we de rouwtaak foto's de namen geven van de rouwtaak? Of had je daar een ander idee voor? Laat me weten dan merge ik het

Goed punt! Ik heb er bewust voor gekozen om de foto's te benoemen op basis van de themanaam, omdat dit korter is en makkelijker werkt bij het ophalen van de data. Zo kan ik de afbeeldingen dynamisch koppelen aan de juiste rouwtaak op basis van de naam die al in de data voorkomt.
zoals hire

<img src="/assets/photos/rouwtaak-{{ exercise.theme }}.svg" alt="{{ exercise.title }} icon">

Laat me weten of je toch liever volledige namen wilt gebruiken, dan pas ik het aan.

@Recss68
Copy link
Copy Markdown
Owner

Recss68 commented May 15, 2025

Goed idee, lijkt me inderdaad handiger als we het zo soepeler kan. Ik merge de request!

@Recss68 Recss68 merged commit addd717 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.

2 participants