Skip to content

feat: redesign introduction page#741

Open
an0ushkah wants to merge 2 commits intomainfrom
feat/intro-page-redesign
Open

feat: redesign introduction page#741
an0ushkah wants to merge 2 commits intomainfrom
feat/intro-page-redesign

Conversation

@an0ushkah
Copy link

Redesigns the introduction page with dynamic component cards pulled from componentCategories.json, grouped by category, with click-through navigation to each component's documentation page.

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

PR Preview Action v1.6.3

🚀 View preview at
https://mParticle.github.io/aquarium/pr-preview/pr-741/

Built to branch gh-pages at 2026-03-04 18:50 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@an0ushkah an0ushkah changed the title Introduction Page redesign feat: redesign introduction page Mar 4, 2026
@@ -0,0 +1,336 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Few thoughts. This is hardcoded, which means AI or human should be aware of this and it a new component is added (not a case for core components, but will be for custom ones) - they will need to update this file. Second, let’s drop variants. I really don’t know if it’s meaningful for consumers. I think what treated as variants here are all properties that component has. But we don’t use a lot of them. And in some cases we even want to prevent usage of some. Think of a buttons. We have type, we have size, we have shape. This is straightforward. But we also have ghost true or false, icon, onClick function. It will count it all. That’s why button has 39 variants. It will be more confusing. So I vote for getting rid of it for now.

]
},
{
"name": "Layout",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let’s exclude layout from here

]
},
{
"name": "Other",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Other and not prod ready are also not needed here

return `${window.top.location.origin}/aquarium/`
}
return `${window.top.location.origin}/`
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Note to self: review this

}

const previewContainerStyle = {
background: '#F5F5F5',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ugh. We should try to use design tokens instead of hardcoded values

}

return (
<a
Copy link
Collaborator

Choose a reason for hiding this comment

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

We not using card component?

Simple, flexible, and reliable—this library helps you focus on building great experiences without
worrying about the basics.
</p>
</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ugh. I updated Claude to not use raw html and prefer our components. But cursor is not aware 😭

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