feat: reorder API page cards and remove duplicate namespace filter#388
Conversation
Move Providers/Consumers cards to top and About/Relations to bottom on API entity page, remove unused Links card. Remove duplicate EntityNamespacePicker from API listing page (already in table header). Signed-off-by: Kavith Lokuhewage <kaviththiranga@gmail.com>
📝 WalkthroughWalkthroughThis pull request removes the EntityNamespacePicker filter component from the CustomApiExplorerPage and reorganizes the card layout in EntityPage's apiPage section, moving EntityProvidingComponentsCard and EntityConsumingComponentsCard into the top overview area while removing the EntityLinksCard from that section. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
packages/app/src/components/catalog/EntityPage.tsx (1)
487-494: Missingspacingandxsprops may cause layout inconsistencies.The nested Grid container lacks
spacing={3}(causing no gap between the two cards), and the inner items lackxs={12}for mobile responsiveness. Other card pairs in this file (e.g., lines 495-504) follow a consistent pattern.♻️ Suggested fix for consistent layout and responsiveness
- <Grid container item md={12}> - <Grid item md={6}> + <Grid container item md={12} spacing={3}> + <Grid item md={6} xs={12}> <EntityProvidingComponentsCard /> </Grid> - <Grid item md={6}> + <Grid item md={6} xs={12}> <EntityConsumingComponentsCard /> </Grid> </Grid>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@packages/app/src/components/catalog/EntityPage.tsx` around lines 487 - 494, The nested Grid container wrapping EntityProvidingComponentsCard and EntityConsumingComponentsCard is missing spacing and the inner Grid items are missing responsive xs props; update the parent Grid (the container with md={12}) to include spacing={3} and add xs={12} to each inner Grid item that wraps EntityProvidingComponentsCard and EntityConsumingComponentsCard to match the other card pairs and ensure mobile responsiveness and consistent gaps.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@packages/app/src/components/catalog/EntityPage.tsx`:
- Around line 487-494: The nested Grid container wrapping
EntityProvidingComponentsCard and EntityConsumingComponentsCard is missing
spacing and the inner Grid items are missing responsive xs props; update the
parent Grid (the container with md={12}) to include spacing={3} and add xs={12}
to each inner Grid item that wraps EntityProvidingComponentsCard and
EntityConsumingComponentsCard to match the other card pairs and ensure mobile
responsiveness and consistent gaps.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 5d015203-4f66-49ac-86b1-b675431b4ad2
📒 Files selected for processing (2)
packages/app/src/components/catalog/CustomApiExplorerPage.tsxpackages/app/src/components/catalog/EntityPage.tsx
💤 Files with no reviewable changes (1)
- packages/app/src/components/catalog/CustomApiExplorerPage.tsx
Move Providers/Consumers cards to top and About/Relations to bottom on
API entity page, remove unused Links card. Remove duplicate
EntityNamespacePicker from API listing page (already in table header).
Before:


After:


Summary by CodeRabbit