Closed kohta9521 closed 4 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
blog-site-kohta | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 9, 2024 4:46am |
[!WARNING]
Review failed
The pull request is closed.
This set of updates focuses on refining and expanding the frontend components. The FooterMenuMain.stories.tsx
file introduces Storybook metadata, enhancing development workflow. The ArticleCard.tsx
and ArticleList.tsx
components now use a string-based key
prop for better data handling. In the CSS, limiting the height of the .tagsBox
and .tag
elements enhances layout consistency.
File Path | Change Summary |
---|---|
.../FooterManuMain.stories.tsx |
Introduces Storybook metadata and a default story for FooterMenuMain component. |
.../ArticleCard.tsx |
Changes key prop type from number to string . |
.../styles/ArticleBlockCard.module.scss |
Adds max-height: 32px to .tagsBox and .tag classes. |
.../ArticleList.tsx |
Updates key attribute in ArticleCard from static value to blog.title . |
In the land of code, where components abide,
AFooterMenu
tale, with stories and pride. 📚
Article cards now with keys of string,
Enhanced layouts that make tags sing.
A rabbit’s hop in CSS's light,
Max-height for tags, a charming sight! 🌿🐇💻
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
New Features
FooterMenuMain
component, allowing easy visualization and testing.Bug Fixes
key
prop type in theArticleCard
component, changing it fromnumber
tostring
for better key handling.Style
max-height
property to the.tagsBox
and.tag
classes inArticleBlockCard
to improve layout and presentation.Refactor
ArticleList
component to dynamically set thekey
attribute inArticleCard
usingblog.title
, enhancing list rendering performance.