dashboard: hide table columns, drop terminal paste button, loading typewriter dots#847
Conversation
…n, loading typewriter dots - Boxes table (desktop): hide Created + Resource Quota columns (kept on the mobile card). - Terminal: remove the paste button - native Cmd/Ctrl+V already pastes into the terminal; drop the now-dead pasteIntoTerminalIframe util. - LoadingFallback: bigger terminal boot screen + typewriter ellipsis (1->2->3 dots, looping).
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe dashboard table drops the Resource Quota and Created columns on desktop, the loading fallback now animates its ellipsis with cycling dots, the terminal frame removes clipboard paste handling, and dashboard page height calculations switch to a 60px base. ChangesDashboard UI updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed: dependency version conflict. Check your lock file or package.json. 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 |
…o the 60px header The shell's --app-content-height subtracted 3.5rem (56px) while the header is h-[60px], so the content area was 4px taller than the viewport -> a persistent document scrollbar stacked on top of the inner content scroll (two scrollbars). Match the offset to 60px, consistent with the pages' own h-[calc(100svh-60px)].
Follow-up tweaks after testing #845 (which is now merged). Frontend-only, 4 files.
pasteIntoTerminalIframeutil.Verified: dashboard build + targeted eslint clean.
Summary by CodeRabbit