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

Remove the heart icon button to add the view as a favorite from the top bar #8769

Merged
merged 5 commits into from
Dec 3, 2024

Conversation

ehconitin
Copy link
Contributor

closes #8546

@Bonapara please check the behaviour, if this is what you were looking for! ;)

isFavoriteFolderEnabled && (
<MenuItem
LeftIcon={IconHeart}
text={'Favorite picker'}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Bonapara I was not sure what to call this :), for now I have named the menu item Favorite picker

@ehconitin ehconitin requested a review from Bonapara November 27, 2024 08:44
@Bonapara
Copy link
Member

Hey @ehconitin!

I cannot see the add/remove from favorites button in the view switcher 🤔

CleanShot.2024-11-27.at.10.11.16.mp4

@ehconitin
Copy link
Contributor Author

@Bonapara
The feature flag must be disabled on your end. Could you make sure IsFavoriteFolderEnabled and IsFavoriteFolderEntityEnabled is enabled :) Thanks!

@Bonapara
Copy link
Member

@ehconitin we would like the view to be added directly to the favorite root level when we press "Add to Favorites". When a view is already part of favorites, the CTA should be called "Manage Favorite".

Also, could you display the "Favorite" menu to the right of the 3 dots menu instead of on the left?

@martmull
Copy link
Contributor

Functional comments:

  • wrong left margin before the heart Icon
    image
  • dropdown should be displayed above the other onces, not replacing them
    image

Comment on lines +107 to +133
dropdownComponents={(() => {
switch (viewPickerMode) {
case 'list':
return <ViewPickerListContent />;
case 'favorite-folders-picker':
return (
isFavoriteFolderEnabled && <ViewPickerFavoriteFoldersDropdown />
);
case 'create-empty':
case 'create-from-current':
return (
<>
<ViewPickerContentCreateMode />
<ViewPickerContentEffect />
</>
);
case 'edit':
return (
<>
<ViewPickerContentEditMode />
<ViewPickerContentEffect />
</>
);
default:
return null;
}
})()}
Copy link
Contributor

Choose a reason for hiding this comment

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

nice

@martmull martmull merged commit 9a65e80 into twentyhq:main Dec 3, 2024
17 checks passed
Copy link

github-actions bot commented Dec 3, 2024

Thanks @ehconitin for your contribution!
This marks your 64th PR on the repo. You're top 1% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter

Contributions

mdrazak2001 pushed a commit to mdrazak2001/twenty that referenced this pull request Dec 4, 2024
…op bar (twentyhq#8769)

closes twentyhq#8546 

@Bonapara please check the behaviour, if this is what you were looking
for! ;)
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.

Remove the heart icon button to add the view as a favorite from the top bar
5 participants