Skip to content

fix: members table header#1413

Merged
rohilsurana merged 1 commit intomainfrom
fix-title
Feb 24, 2026
Merged

fix: members table header#1413
rohilsurana merged 1 commit intomainfrom
fix-title

Conversation

@rohanchkrabrty
Copy link
Contributor

Summary

This PR fixes a styling bug where the table header is not correctly positioned

@rohanchkrabrty rohanchkrabrty self-assigned this Feb 24, 2026
@vercel
Copy link

vercel bot commented Feb 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
frontier Ready Ready Preview, Comment Feb 24, 2026 6:32am

@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d1d7f2f and 32e250d.

📒 Files selected for processing (1)
  • web/sdk/react/components/organization/members/members.module.css

📝 Walkthrough

Summary by CodeRabbit

  • Style
    • Adjusted vertical positioning of table headers in the organization members view for improved visual alignment.

Walkthrough

A CSS property addition to the .tableHeader class in the organization members module. The top: -1px value is introduced alongside the existing z-index: 1 declaration to adjust the vertical positioning of the table header element upward by one pixel.

Changes

Cohort / File(s) Summary
CSS Positioning
web/sdk/react/components/organization/members/members.module.css
Added top: -1px property to .tableHeader selector to shift header position upward by 1 pixel.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes


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.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 22339500016

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 39.848%

Totals Coverage Status
Change from base Build 22336016583: 0.0%
Covered Lines: 13595
Relevant Lines: 34117

💛 - Coveralls

@rohilsurana rohilsurana merged commit 513d0c2 into main Feb 24, 2026
8 checks passed
@rohilsurana rohilsurana deleted the fix-title branch February 24, 2026 06:40
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.

3 participants