fix: widen Pull Requests linked-issues column#156
Conversation
|
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 Pull Requests table's Issues column width is standardized using a new constant (88px) to prevent content truncation. The table minimum width is adjusted accordingly, and the constant is applied consistently across the column header, loading skeleton, and all row cells. ChangesIssues Column Width Standardization
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 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 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
src/app/pulls/page.tsxESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox. 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 |
c5c8534 to
20c1d86
Compare
Give the Issues column enough fixed width for RelatedIssuesCell chips and bump table min-width so the column is not clipped (MkDev11#108). Fixes MkDev11#108
20c1d86 to
0fc7327
Compare
|
Please add screenshots |
Summary
Fixes clipping of the Issues column on
/pullswhereRelatedIssuesCellchips were truncated undertable-layout: fixed.ISSUES_COL_WIDTH(88px) on the header and body cellsminWidthby the same delta so horizontal scroll reveals the full columnRelated Issues
Fixes #108
Type of Change
Testing
pnpm run lint(CI)pnpm build(CI)Checklist
Summary by CodeRabbit