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

Activity Library v2: favourite activities UI #9114

Closed
7 tasks
nickoferrall opened this issue Nov 3, 2023 · 15 comments · Fixed by #9680
Closed
7 tasks

Activity Library v2: favourite activities UI #9114

nickoferrall opened this issue Nov 3, 2023 · 15 comments · Fixed by #9680
Assignees

Comments

@nickoferrall
Copy link
Contributor

Figma designs: https://www.figma.com/file/PmE9A2ryODVzFPjZFJn7qi/Parabol-Activity-Library?type=design&node-id=1036-28426&mode=design&t=EUnFT9A69gLQn38c-0

Screenshot 2023-11-03 at 13 10 37
Screenshot 2023-11-03 at 13 08 30
Screenshot 2023-11-03 at 13 11 24
Screenshot 2023-11-03 at 13 11 17

AC

  • Add favourites section next to "Quick Start" with a heart emoji
  • Add empty state
  • Add heart button to activity cards
  • Add heart button to selected template
  • Move heart button to toolbar on mobile
  • Clicking the heart icon updates the UI but doesn't do anything more than that. Logic will be handled in a subsequent PR
  • Include tooltip when hovering over heart button
@nickoferrall
Copy link
Contributor Author

@enriquesanchez just want to check whether the heart button would appear at the bottom right of all activity cards? I see the design below doesn't have them:

Screenshot 2023-11-03 at 13 17 01

I'd imagine it would show the heart button in red. Clicking it would remove it from the favourites section?

@enriquesanchez
Copy link
Contributor

@nickoferrall Ah yes, you are correct! The heart should be there for any activity you have favorited. Apologies, the heart in the card idea came after I worked on those mockups.

One thing that was left inconclusive and we need to revisit as part of this effort is the fact that the current hover interaction for cards shows the activity's contents:

Captura de pantalla 2023-11-06 a la(s) 5 39 22 p m

This hover effect clashes with the action of favoriting an activity, since both trigger on hover. This is something we can look into together and find a better solution, @acressall would you be interested? I can't remember who nor where, but it was suggested to use the side-sheet pattern for showing activity details instead. I think there may be potential for a better solution there.

@nickoferrall
Copy link
Contributor Author

Makes sense, thanks!

@enriquesanchez I saw this design for the hover: #9116

Would that work with the heart action?

@enriquesanchez
Copy link
Contributor

Would that work with the heart action?

I think it'll be tricky, since the tooltip would sit on top of the heart toggle. This is one we need to pause and think more about. One immediate solution I can think of is to move the heart toggle to the left (bottom or top), since that's an are where it'll be unlikely for the tooltip to cover.

This is one where ideally we can sit together and find a better solution. Maybe Alicia, you and I can schedule some sync time?

@acressall
Copy link
Contributor

@enriquesanchez feel free to put some time in my calendar!

@nickoferrall
Copy link
Contributor Author

Perhaps the tooltip could open to the left instead of the right? That way, the heart could remain in the same place, and it wouldn't be covered by the tooltip?

Also, happy to chat sync!

@nickoferrall
Copy link
Contributor Author

@enriquesanchez & @acressall just checking if this came up in the design meeting? Perhaps I could implement the tooltip opening to the left as a starting point and it could be changed later on?

@enriquesanchez
Copy link
Contributor

@nickoferrall We didn't review this, but I prefer the heart to the left (this also avoids conflicts with the Premium tag placement) and the tooltip opening as expected to the right.

I still want to look at the side sheet pattern I mentioned above but I need a little more time. I will try to carve some time tomorrow to look at it.

@nickoferrall
Copy link
Contributor Author

@enriquesanchez & @acressall sounds good. I think we should ship the activity library card UI (this PR #9168) asap, but we'll need to have a tooltip or some UI that shows the template prompts first.

Perhaps I can implement the tooltip UI that you designed (see below), we ship the tooltip and card UI to production, and then I pause development until the designs that factor in the heart UI & premium label are ready?

image

@enriquesanchez
Copy link
Contributor

but we'll need to have a tooltip or some UI that shows the template prompts first.

I think it's worth trying the tooltip. We did have one concern around timing between hover and displaying the tooltip, we want to make sure it's not annoying, as in you see tooltips all over the place as you move the pointer around. Maybe a short delay helps with this.

Perhaps I can implement the tooltip UI that you designed (see below), we ship the tooltip and card UI to production, and then I pause development until the designs that factor in the heart UI & premium label are ready?

I like this plan 👍

@jordanh
Copy link
Contributor

jordanh commented Mar 15, 2024

I feel it would be safe to release the AL to everybody, and then follow on with this. Will prioritize accordingly to the backlog now

@nickoferrall nickoferrall self-assigned this Apr 23, 2024
@nickoferrall
Copy link
Contributor Author

Hey @ackernaut, for the empty state, is there a good way to export the image from figma? When I click on the image and click export PNG, I see this:

Screenshot 2024-04-24 at 20 45 18

If not, I can work on recreating it with CSS.

Also, in the design, the heart emoji is white with a purple background, but this is the background colour for "Quick Start", so I think we need something else. Maybe the heart could be red? Not sure about the background colour.

Screenshot 2024-04-24 at 20 47 42

Screenshot 2024-04-24 at 20 48 44

@ackernaut
Copy link
Member

@nickoferrall thanks for asking!

Re: fav background image

It looks bad in preview because of the transparency. It should look good over a white background in the app. Here are a couple of files where I rasterized the image in Figma. (2x and 3x sizing)

activity-library-favs-empty-2x

activity-library-favs-empty-3x

That makes it simple enough to drop in but the text isn’t HTML. We could consider just using the background shape and making the heart, circle, and text in code.

al-favs-background-vector-2x

al-favs-background-vector-3x

@ackernaut
Copy link
Member

Re: favorites nav tab

I like it how it was designed, I think it’s cleaner. They shouldn’t both be selected at the same time. I think of quick start and favorites as functional tabs whereas the others are colorized category tabs. Perhaps it makes some sense to have the first two the same selected color. I’d recommended going with the design. I appreciate the attention to detail and feedback.

@nickoferrall
Copy link
Contributor Author

Thank you, Terry! That's very helpful 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Status: No status
Development

Successfully merging a pull request may close this issue.

5 participants