-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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(): Onboarding Checklist #2863
feat(): Onboarding Checklist #2863
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
return completed; | ||
}); | ||
|
||
watch(checkListCompleted, completed => { |
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.
Instead of auto hiding it on completion we could have a share on twitter button
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.
I think it could be a good idea. I think it should be also discussed with @bonustrack.
It will ask some UX thinking about how to handle the closing of this panel.
const hasVoted = ref(false); | ||
const hasCreatedProposal = ref(false); | ||
|
||
const onboardingChecklist = ref([ |
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.
It could be hard for new users figuring out how to complete each list item. Maybe each item should be collapsable with additional links and information. @ManuBernard do you have a designer with who we could try some things out?
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.
I have followed the design provided by @bonustrack :
https://www.figma.com/file/GS2AS8G3YCYjeH1dFTFpIn/Snapshot---onboarding-section?node-id=0%3A1
Not sure how to move forward, maybe you should talk together and see if we want to improve this in this PR or create a new one later?
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.
Oh I see. Yeah I wasn't aware of this design sorry for the confusion :)
40dab16
to
8975a35
Compare
Reopened this PR from a snapshot branch here #2941 |
Changes in this PR:
Add onboarding checklist to Timeline