Skip to content

Add keys to columns and change max width class#4

Merged
RhysSullivan merged 3 commits intoRhysSullivan:mainfrom
viralcodex:main
Feb 23, 2026
Merged

Add keys to columns and change max width class#4
RhysSullivan merged 3 commits intoRhysSullivan:mainfrom
viralcodex:main

Conversation

@viralcodex
Copy link
Contributor

@viralcodex viralcodex commented Feb 23, 2026

@RhysSullivan
There was an issue in dev mode where the key prop was missing in the component. Also fixed a TailwindCSS warning for best practise

Wrap the PR, issues, and repos columns in divs with explicit key props to avoid React key warnings when rendering the three-column grid. Also replace the container Tailwind class max-w-[1600px] with max-w-400 to adjust the layout width.
@vercel
Copy link

vercel bot commented Feb 23, 2026

@viralcodex is attempting to deploy a commit to the Rhys's Projects Team on Vercel.

A member of the Team first needs to authorize it.

return (
<div className="h-full overflow-y-auto bg-dotgrid">
<div className="mx-auto max-w-[1600px] px-4 py-4 md:px-6 md:py-5">
<div className="mx-auto max-w-400 px-4 py-4 md:px-6 md:py-5">
Copy link
Owner

Choose a reason for hiding this comment

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

why'd this change?

Copy link
Contributor Author

@viralcodex viralcodex Feb 23, 2026

Choose a reason for hiding this comment

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

warning from Tailwind in VSCode that it can be written like this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

useCanonicalClasses is the suggestion from Tailwind

@RhysSullivan RhysSullivan merged commit f28ff6a into RhysSullivan:main Feb 23, 2026
1 check failed
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