cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

Published 20 hours ago • verified publisher icon calcom

/onboarding - UI issues & small improvements

Open ciaranha opened this issue 3 years ago • 3 comments

General

  • [ ] There's still a long view of the app before onboarding starts.. can we do anything about that? It's not smooth. @zomars https://github.com/calcom/cal.com/issues/4383
  • [ ] All Steps: Change "Next Step" in button to "Continue"
  • [ ] Something off with a lot of the buttons. Are we using the right components?

Welcome 1/4

  • [ ] The first step is disabled but the style looks weird. Let's just make it editable. https://github.com/calcom/cal.com/issues/4382 Image

Connect calendar - No calendar connected 2/4

  • [ ] Button text looks too bold Image

Connect calendar - Calendar connected 2/4

  • [ ] We should toggle on at least the first calendar in the list here if not all? https://github.com/calcom/cal.com/issues/4384
  • [ ] Change the mt-7 above the "You can add more...." copy to an mt-4. It's way too big right now.
  • [ ] THe edit button currently does nothing. Change it to delete so they can remove and add another one. https://github.com/calcom/cal.com/issues/4385

Image

Availability step 3/4 Created seperate cards. https://github.com/calcom/cal.com/issues/4380 & https://github.com/calcom/cal.com/issues/4381

  • [ ] Would it make more sense to start on a Monday? If we're going to default to Sunday being unavailable it makes more sense imo.
  • [ ] Something wrong with this button visually. Text colour is wrong. Button should say "Copy to all" and Tooltip copy should say "Copy times to all days". If there's issue with spacing to get that button copy right, make the outer padding on all the cards 24px instead of 32px.
  • [ ] Plus button icon colour is off. Are we using the right component here?
  • [ ] Can we do shorthand for days here? "Mon" etc. or that complicated with translations? Image

Last step 4/4

  • [ ] Upload button text size & copy is wrong - https://github.com/calcom/cal.com/issues/4343
  • [ ] Remove that Title

Image

ciaranha avatar Sep 10 '22 11:09 ciaranha

good ticket but definitely not a quick win. i feel like this should be multiple tickets / PRs too

PeerRich avatar Sep 10 '22 15:09 PeerRich

I'm handling the step3 issues due to shared component with availability single view.

emrysal avatar Sep 12 '22 12:09 emrysal

Left some comments about the onboarding flash in #4383

zomars avatar Sep 12 '22 20:09 zomars

Hey @Jaibles, how are you? I would like your attention on this issue.

Currently steps 1 (welcome) and 2 (connect calendar - No calendar connected) are fully resolved.

Step 3 (availability) is practically resolved, just missing the last check, but I just opened an issue that covers this issue.

Here are the prints in question: Captura de tela de 2022-10-13 16-18-50 Captura de tela de 2022-10-13 16-19-25 Captura de tela de 2022-10-13 16-19-41

matheusmuniz03 avatar Oct 13 '22 19:10 matheusmuniz03

Now steps 2 (Connect Calendar - Calendar connected) are linked in the issue (I commented my solution in the issue) and 4 (Last step) just need to remove the Title. Do you want to close this entire issue and just open another one talking about removing the title or do you think this is good? @Jaibles

matheusmuniz03 avatar Oct 13 '22 19:10 matheusmuniz03