-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: Fix table columns when hidden are not greyed out in preview mode #35729
chore: Fix table columns when hidden are not greyed out in preview mode #35729
Conversation
WalkthroughThe recent changes enhance the styling of the Changes
Assessment against linked issues
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- app/client/src/widgets/wds/WDSTableWidget/component/styles.module.css (2 hunks)
Additional comments not posted (2)
app/client/src/widgets/wds/WDSTableWidget/component/styles.module.css (2)
81-83
: Great job on enhancing visual feedback for hidden column headers!The addition of setting opacity for column headers with the
aria-hidden
attribute effectively communicates their hidden status. This aligns well with the PR's objective to improve user experience by visually indicating inactive elements.
158-160
: Excellent work on ensuring consistency for hidden table cells!Applying opacity to cells with the
aria-hidden
attribute provides a clear visual cue for their inactive status, enhancing user understanding of hidden columns in preview mode. This change is well-aligned with the PR's goals.
Fixes #34852
Grey out the header cells and row cells when the column is set invisible.
Before:
After:
/ok-to-test tags="@tag.Widget"
Tip
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10422331593
Commit: 045a3e7
Cypress dashboard.
Tags:
@tag.Widget
Spec:
Fri, 16 Aug 2024 16:28:06 UTC
Summary by CodeRabbit
WDSTableWidget
by improving opacity styles for disabled table headers and cells.