Skip to content

ref(groupingInfo): move feedback button inline with groupInfoSummary#96908

Merged
shayna-ch merged 5 commits intomasterfrom
shayna-ch/feedback-inline-with-grouping-summary
Aug 4, 2025
Merged

ref(groupingInfo): move feedback button inline with groupInfoSummary#96908
shayna-ch merged 5 commits intomasterfrom
shayna-ch/feedback-inline-with-grouping-summary

Conversation

@shayna-ch
Copy link
Copy Markdown
Member

@shayna-ch shayna-ch commented Jul 31, 2025

Move Give Feedback button to be more inline with the groupingInfoSummary component now that selectGroupingConfig component is gone(#96655). Changed size to xs.

Created feedbackComponent for consistency across old and new UI.

Before:
image

After:
image

@shayna-ch shayna-ch self-assigned this Jul 31, 2025
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 31, 2025
@shayna-ch shayna-ch changed the title ref(groupingInfo): move feedback button more inline with groupInfoSum… ref(groupingInfo): move feedback button inline with groupInfoSummary Aug 1, 2025
@shayna-ch shayna-ch requested review from a team and lobsterkatie August 1, 2025 21:32
@shayna-ch shayna-ch marked this pull request as ready for review August 1, 2025 21:34
cursor[bot]

This comment was marked as outdated.

@scttcper
Copy link
Copy Markdown
Member

scttcper commented Aug 1, 2025

can we switch it to a xs button

@scttcper
Copy link
Copy Markdown
Member

scttcper commented Aug 1, 2025

might need to remove align-items center
image

@shayna-ch shayna-ch force-pushed the shayna-ch/feedback-inline-with-grouping-summary branch from 89f1b98 to 071a121 Compare August 4, 2025 15:41
cursor[bot]

This comment was marked as outdated.

@shayna-ch shayna-ch merged commit 20ba048 into master Aug 4, 2025
49 checks passed
@shayna-ch shayna-ch deleted the shayna-ch/feedback-inline-with-grouping-summary branch August 4, 2025 20:10
@github-actions github-actions bot locked and limited conversation to collaborators Aug 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants