Skip to content
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

fix: group by sort options should have a checkmark when selected #9038

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

magrinj
Copy link
Member

@magrinj magrinj commented Dec 12, 2024

Fix #8966

This PR is adding a checkmark beside the selected sort option in the Options dropdown menu.

Screenshot 2024-12-12 at 12 03 18 PM

@magrinj magrinj marked this pull request as ready for review December 12, 2024 11:04
@magrinj magrinj enabled auto-merge (squash) December 12, 2024 11:04
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This PR adds visual feedback by displaying a checkmark next to the selected sort option in the record group dropdown menu.

  • Replaced MenuItem with MenuItemSelect in /packages/twenty-front/src/modules/object-record/object-options-dropdown/components/ObjectOptionsDropdownRecordGroupSortContent.tsx
  • Updated state management to use useRecoilComponentStateV2 for accessing both current sort value and setter
  • Added selected prop to MenuItemSelect components to highlight active sort option

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

@martmull martmull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@magrinj magrinj merged commit 5f2a39d into main Dec 12, 2024
19 checks passed
@magrinj magrinj deleted the fix/group-by-sort-options branch December 12, 2024 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Record group options current sort not selected
2 participants