Handle singular and plural forms for word, minute and post counts #161
Conversation
…rd.astro and [...slug].astro Updated the word and minute count display in both PostCard.astro and [...slug].astro to correctly handle singular forms when the count is 1. This ensures proper grammar by dynamically selecting the appropriate form based on the count value.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Add conditional logic to display 'postCount' for single post and 'postsCount' for multiple posts in the group list.
Marcos03BR
changed the title
Marcos03BR
changed the title
src/components/ArchivePanel.astro
Outdated
| @@ -68,18 +68,22 @@ function formatTag(tag: string[]) { | |||
| <div class="card-base px-8 py-6"> | |||
| { | |||
| groups.map(group => ( | |||
| <div> | |||
| <div key={group.year}> | |||
There was a problem hiding this comment.
Could you please explain the purpose of adding this attribute?
There was a problem hiding this comment.
This is my first project using Astro, so I’m still learning. In React, it’s considered a good practice to use key when generating lists with map to optimize rendering and avoid potential issues, as it helps uniquely identify each element. Since Astro has some similarities, I thought applying it here might be beneficial, although I’m not sure if it’s strictly necessary. If you feel it doesn’t add value in this case, feel free to remove it, as it works the same without it.
There was a problem hiding this comment.
I guess this is unnecessary in Astro since there is no dynamic element updating. If you have time, could you help me remove these attributes? I'm worried that I might forget why they are here in the future.🥲
Added link to the Spanish readme
src/components/ArchivePanel.astro
Outdated
| aria-label={post.data.title} | ||
| class="group btn-plain block h-10 w-full rounded-lg hover:text-[initial]" | ||
| <a | ||
| key={post.slug} |
|
Done! Sorry for taking so long |
README.md
Outdated
There was a problem hiding this comment.
Apologies for the delay. Could you please revert this deletion? Since it causes a conflict.
Updated the word and minute count display in both PostCard.astro and [...slug].astro to correctly handle singular forms when the count is 1. This ensures proper grammar by dynamically selecting the appropriate form based on the count value.