changed recent issues, recent releases to 2 column layout#1090
changed recent issues, recent releases to 2 column layout#1090arkid15r merged 4 commits intoOWASP:mainfrom
Conversation
Summary by CodeRabbit
WalkthroughThe pull request modifies the Changes
Suggested reviewers
Tip ⚡🧪 Multi-step agentic review comment chat (experimental)
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
frontend/src/components/CardDetailsPage.tsx (1)
91-120: Grid Layout Container ConfigurationThe replacement of the fragment with a
<div>that usesgrid-cols-2 gap-4 lg:gridmeets the PR objective by arranging the "Recent Issues" and "Recent Releases" in two columns on large screens. Please verify that the intended responsive behavior is achieved:
- On large screens, the
lg:gridclass enables grid display with two columns as expected.- On smaller screens, the absence of the
gridclass means the layout will default to a block display (i.e., single column).If the design should always use a grid layout regardless of screen size, consider modifying the container’s classes to something like
grid grid-cols-2 gap-4rather than limiting grid display to large screens.
arkid15r
left a comment
arkid15r
left a comment
There was a problem hiding this comment.
Looks good.
Could you also take care of 1 comments cases in issues by wrapping it into pluralize?
Pushed this change |
|
* changed recent issues, recent releases to 2 column layout * used pluralize for comments * Update code --------- Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
2 participants
Resolves #1089
Changed project details page recent issues / recent releases to a 2 column layout