fix(ui): polish console surfaces#2204
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR refines the design system by updating DaisyUI theming, Tailwind tokens (including letter-spacing to 0), and applying those changes across navigation, tabs, tables, and settings. It simultaneously enhances accessibility by adding ARIA labels to action buttons and current-state indicators to tabs. ChangesAccessibility & Design System Updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Merging this PR will not alter performance
Comparing 
Footnotes
|
riderx
force-pushed
the
codex/ui-polish-tweaks
branch
from
d45be3c to
533b4a5
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
riderx
force-pushed
the
codex/ui-polish-tweaks
branch
2 times, most recently
from
43a61ca to
fe49277
Compare
riderx
force-pushed
the
codex/ui-polish-tweaks
branch
from
fe49277 to
349425d
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/pages/admin/dashboard/replication.vue (1)
177-179: ⚡ Quick winHardcoded English strings should use i18n translation keys.
Lines 178, 196, and several table headers/labels use hardcoded English text instead of
t('key')calls. As per coding guidelines, use translation keys only and add missing keys tomessages/en.json.Examples:
- Line 178:
"Logical replication slot lag monitoring"→t('replication-slot-lag-monitoring')- Line 196:
'Refreshing...'/'Refresh'→t('refreshing')/t('refresh')Also applies to: 196-196
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/admin/dashboard/replication.vue` around lines 177 - 179, Replace hardcoded UI strings in the replication Vue template with i18n calls: change "Logical replication slot lag monitoring" to t('replication-slot-lag-monitoring'), change "Refreshing..." and "Refresh" to t('refreshing') / t('refresh') and similarly convert any table headers/labels to t(...) using descriptive keys; update messages/en.json with the new keys and English values, and ensure the component imports/uses the i18n composable (t) used elsewhere in this project so the template calls resolve correctly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/pages/admin/dashboard/replication.vue`:
- Around line 111-112: The thrown Error message is misleading because this path
now requires a Supabase session; update the error thrown when checking
session?.access_token so it no longer mentions the "replication secret" and
instead clearly states that a valid Supabase session/access token is required
(e.g., "No Supabase session available; replication requires a valid access
token"). Locate the check using session?.access_token in replication.vue and
replace the error text accordingly.
---
Nitpick comments:
In `@src/pages/admin/dashboard/replication.vue`:
- Around line 177-179: Replace hardcoded UI strings in the replication Vue
template with i18n calls: change "Logical replication slot lag monitoring" to
t('replication-slot-lag-monitoring'), change "Refreshing..." and "Refresh" to
t('refreshing') / t('refresh') and similarly convert any table headers/labels to
t(...) using descriptive keys; update messages/en.json with the new keys and
English values, and ensure the component imports/uses the i18n composable (t)
used elsewhere in this project so the template calls resolve correctly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: bc629d6c-6b3f-49fe-b77a-570a2806528c
⛔ Files ignored due to path filters (10)
docs/pr-screenshots/ui-polish-tweaks/app-detail-after.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/app-detail-before.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/apps-after.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/apps-before.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/dashboard-after.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/dashboard-before.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/login-after.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/login-before.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/settings-organization-after.pngis excluded by!**/*.pngdocs/pr-screenshots/ui-polish-tweaks/settings-organization-before.pngis excluded by!**/*.png
📒 Files selected for processing (10)
src/components/DataTable.vuesrc/components/Navbar.vuesrc/components/Sidebar.vuesrc/components/TabSidebar.vuesrc/components/Tabs.vuesrc/components/dashboard/ChartCard.vuesrc/layouts/settings.vuesrc/pages/admin/dashboard/replication.vuesrc/pages/settings/organization/index.vuesrc/styles/style.css
# Conflicts: # src/components/Sidebar.vue # src/pages/admin/dashboard/replication.vue
|
KCDaemon
left a comment
KCDaemon
left a comment
There was a problem hiding this comment.
Reviewed the UI-polish diff with focus on behavior risk rather than the screenshot assets. The changes are limited to touch-target sizing, focus/ARIA metadata, table overflow handling, ChartCard spacing, and settings-page button sizing; I did not find a new functional blocker in the touched components.\n\nVerification performed here:\n- passes.\n- Reviewed the code-only diff for DataTable, Navbar, Sidebar, TabSidebar, Tabs, ChartCard, settings layout, organization settings page, and style.css.\n\nThe one i18n cleanup around replication labels is already covered by the existing CodeRabbit comment, so I am not duplicating that thread.
3 participants
Summary (AI generated)
docs/pr-screenshots/ui-polish-tweaks/.Motivation (AI generated)
Small shared polish changes improve the feel of the Capgo console without redesigning the product: larger touch targets, clearer focus states, less cramped tables, more consistent chart cards, and better settings page density.
Business Impact (AI generated)
The console should feel more stable and easier to operate across dashboard, app, table, and settings workflows. This reduces small UX friction for developers managing releases while keeping the existing Capgo visual language intact.
Screenshots (AI generated)
Test Plan (AI generated)
bun lintbun typecheckbun run buildSummary by CodeRabbit
Accessibility
Style