Skip to content

Conversation

PuruVJ
Copy link
Collaborator

@PuruVJ PuruVJ commented Oct 24, 2024

Closes #617

Copy link

vercel bot commented Oct 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kit-svelte-dev 🔄 Building (Inspect) Visit Preview 💬 Add feedback Oct 25, 2024 0:43am
learn-svelte-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 25, 2024 0:43am
omnisite ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 25, 2024 0:43am
svelte-5-preview 🔄 Building (Inspect) Visit Preview 💬 Add feedback Oct 25, 2024 0:43am
svelte-dev 🔄 Building (Inspect) Visit Preview 💬 Add feedback Oct 25, 2024 0:43am

@bleucitron
Copy link

Is it possible to handle banners without endDate ? i'd like to add one on the french version to warn people the translation is WIP, but i need to specify and endDate to make it appear.

I can set some random date in the future of course, but this could be nice to have.

@bleucitron
Copy link

also, do banners need to always be links ?

(i can open an issue for these 2 points if you'd like)

@PuruVJ
Copy link
Collaborator Author

PuruVJ commented Oct 25, 2024

Is it possible to handle banners without endDate ? i'd like to add one on the french version to warn people the translation is WIP, but i need to specify and endDate to make it appear.

You could add a date for 100 years after now 😄. It can be added to the site, but I see Rich removed the defineBanner function, which couldve dealt with an infinite endDate, so IG endDate is now explicit.

also, do banners need to always be links ?

You could apply an action in your own repo to the Banner component. Allowing it to do more things could make the API very complicated for not much ROI

@Rich-Harris Rich-Harris merged commit 6b4ac05 into main Oct 25, 2024
7 checks passed
@Rich-Harris Rich-Harris deleted the banner-svelte-hack branch October 25, 2024 12:09
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.

Add hackathon banner to the docs

4 participants