Revive purple friday website #632
Conversation
thomcsmits
requested review from
casperboone,
ARBaart,
laurensWe and
vanadie
Deploying dwhdelft-nl with
|
| Latest commit: |
45c34bc
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://bcd8e13b.dwhdelft-nl-u1a.pages.dev |
| Branch Preview URL: | https://thomcmsits-paarsevrijdag.dwhdelft-nl-u1a.pages.dev |
Deploying outsite-nl with
|
| Latest commit: |
45c34bc
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fabe983b.dwhdelft-nl.pages.dev |
| Branch Preview URL: | https://thomcmsits-paarsevrijdag.dwhdelft-nl.pages.dev |
| active.value = index | ||
| } | ||
|
|
||
| watchEffect(() => { |
There was a problem hiding this comment.
Waarom zit dit in een watch? Lijkt constant te blijven 🤔
There was a problem hiding this comment.
- Ik vind het altijd raar dat dit op /resources leeft en daar vervolgens nog een half jaar rondzwerft, moeten we het niet op /paarsevrijdag oid gooien?
- "HH" voelt gek, misschien gewoon uitschrijven?
- Veel "wit"ruimte tussen titel en content
- Tabs pakken niet de volledige ruimte
CleanShot 2024-12-10 at 16 28 46@2x
There was a problem hiding this comment.
Ik zou een refactor naar het gebruik van https://headlessui.com/v1/vue/tabs nice vinden. Snap dat er nu tijdsdruk is, dus misschien voor na het mergen van deze PR
There was a problem hiding this comment.
Ja nice! Idd voor daarna
| <div class="w-full py-6"> | ||
| <section class="container mx-auto space-y-6"> | ||
| <div class="space-y-6"> | ||
| <div class="text-5xl"> | ||
| {{ t('title') }} | ||
| </div> | ||
| <div> | ||
| {{ t('subtitle') }} | ||
| </div> |
There was a problem hiding this comment.
Ik zou dit hele stuk naar de resources.vue gooien, en alleen de tabs hierin laten
There was a problem hiding this comment.
Juist omdat dit het enige wat specific is voor paarse vrijdag, zou ik die voor nu hier laten, omdat je dit hele paarse vrijdag gebeuren weer weg kan halen (/verplaatsen) door alleen die weg te halen
|
Mee eens dat het op /resources niet de meest logische plek is. Vandaar #539 , die stond heel lang op awaiting-content (Dante zou hier een overzicht voor maken) maar die is gesloten omdat de content verwijderd was. Mee eens dat dit beter gesplitst kan worden, maar idd voor na paarse vrijdag. |
No description provided.