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
feat(EMI-1773): update copy for saves page tooltips and modals #13643
feat(EMI-1773): update copy for saves page tooltips and modals #13643
Conversation
src/Apps/CollectorProfile/Routes/Saves/Components/ArtworkListContent.tsx
Outdated
Show resolved
Hide resolved
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.
Looks good! Feel free to merge this and let me handle the merge conflicts in my PR.
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.
This looks good to me.
I think we are missing one more usage of this LockIcon that is replaced with HideIcon - SelectArtworkListItem
that is used to when user selects which list to add saved artwork to. I assume the icon needs to be changed there as well?
src/Apps/CollectorProfile/Routes/Saves/Components/ArtworkListContent.tsx
Show resolved
Hide resolved
Good callout, it was not mentioned in the ticket but we should update it there as well, thanks for flagging! |
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.
👍
The type of this PR is: Feat
This PR solves EMI-1773
Description
Update the "Create a new list" description copy
Replace the lock icon for a hide icon
Update the list header to remove the list privacy (Shared or Private) and move the icon to the title itself
Add tooltips for both icons
Replace the lock icon in the SelectArtworkListItem modal
This will have merge conflicts with #13641, so we should merge that first.
@artsy/emerald-devs