Skip to content

Conversation

@akshay1502
Copy link

@akshay1502 akshay1502 commented Oct 23, 2025

Description

Removed sticky from first header cell and table data cells of first column.

Solves below issue:
#7894

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Improvement (change that would cause existing functionality to not work as expected)
  • Code refactoring
  • Performance improvements
  • Documentation update

Screenshots and Media (if applicable)

Screen.Recording.2025-10-24.at.1.14.28.AM.mov

Test Scenarios

References

Summary by CodeRabbit

  • Style
    • Adjusted spreadsheet header and first-column positioning and stacking behavior. This changes how header and row cells align and overlap during scrolling, reducing previous sticky offset and z-index stacking so cells remain relatively positioned and alter visual stacking order.

@CLAassistant
Copy link

CLAassistant commented Oct 23, 2025

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 23, 2025

Walkthrough

Removed medium-screen sticky positioning and explicit left-offset/z-index classes from a header cell and a first table cell in the spreadsheet layout, reverting those elements to normal relative positioning while keeping other styles intact.

Changes

Cohort / File(s) Summary
Spreadsheet header & row sticky removal
apps/web/core/components/issues/issue-layouts/spreadsheet/spreadsheet-header.tsx, apps/web/core/components/issues/issue-layouts/spreadsheet/issue-row.tsx
Deleted md:sticky (or equivalent medium-breakpoint sticky modifier), left-0, and z-[15] class values from the sticky header cell and first row cell so they no longer use sticky positioning or explicit stacking; other classes preserved.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nudged the sticky seam away,
Left the rows to hop and play,
No z-stack towers, calm and light,
A softer spreadsheet, simple sight. 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The pull request title "fix: removed sticky from first column header cell and data cell" directly corresponds to the actual changes made. The title clearly and specifically describes the primary modification: removing sticky positioning from both the header cell and data cells of the first column. The title is concise, uses the conventional "fix:" prefix, and provides sufficient specificity that a teammate can understand the main change without ambiguity.
Description Check ✅ Passed The pull request description follows the required template structure and includes most critical sections: a clear description of changes (removing sticky positioning from the first column), the type of change correctly marked as "Bug fix," a visual asset (screenshot/video) demonstrating the fix, and a reference to the related issue. The Test Scenarios section is left empty, which is a minor gap, but the description is otherwise complete and informative with concrete details about what was changed and why.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 347b6ad and 5213e13.

📒 Files selected for processing (2)
  • apps/web/core/components/issues/issue-layouts/spreadsheet/issue-row.tsx (1 hunks)
  • apps/web/core/components/issues/issue-layouts/spreadsheet/spreadsheet-header.tsx (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • apps/web/core/components/issues/issue-layouts/spreadsheet/spreadsheet-header.tsx
  • apps/web/core/components/issues/issue-layouts/spreadsheet/issue-row.tsx

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@akshay1502 akshay1502 force-pushed the fix/work-item-table-header-7894 branch from 347b6ad to 5213e13 Compare October 24, 2025 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants