Skip to content

New Feat: Blog Post#41

Open
DavidTrujillo123 wants to merge 3 commits into
mainfrom
feat/feed
Open

New Feat: Blog Post#41
DavidTrujillo123 wants to merge 3 commits into
mainfrom
feat/feed

Conversation

@DavidTrujillo123
Copy link
Copy Markdown
Collaborator

New:

  • Add new page: feed.
  • Add new components: Chip, GridCards.
  • Add publications on navbar.

Comment thread src/components/molecules/Chip.astro Outdated
---
<div class="w-full mb-2 cursor-default">
{ texts.map((text, index) => (
<p class={`inline-block p-2 text-sm rounded-xl ${onHovermyVariants[variantKeys[index % variantKeys.length]]}`}><span class={`${myVariants[variantKeys[index % variantKeys.length]]} border-`}>#</span>{text}</p>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try to make this more atomic, extract the chip in only one component and you could make another component for the chips list

import CardFeed from "../../components/organisms/CardFeed.astro";
import Divider from "../../components/atoms/Divider.astro";

const dataExample = [
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try to add this data as a prop in order to make it more reusable

@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 23, 2024

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

Name Status Preview Comments Updated (UTC)
spaceapps ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 23, 2024 5:12pm

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