-
Notifications
You must be signed in to change notification settings - Fork 327
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
for #20: start onboarding UI in pop-up #29
Conversation
@@ -40,6 +40,33 @@ function showContainerTabs(containerId) { | |||
hideorshowIcon.src = CONTAINER_HIDE_SRC; | |||
} | |||
|
|||
if (localStorage.getItem('onboarded2')) { |
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.
@jonathanKingston - I'm really not happy with this logic for controlling the shown/hidden state of the onboarding vs. container panels. I'll keep poking at this code to improve it.
@TanviHacks do you mean each of those elements needs its own onboarding tutorial/panel? |
Deleted extra comments. Assigning to @jonathanKingston for review & spot-check. Note: This depends on us having Firefox 52 ContextualIdentityService module, so we need to get that sorted. |
fdf5c95
to
8eb52c7
Compare
Rebased this branch on @bakulf and @jonathanKingston's latest commits in master. |
Work-in-progress; still needs: