feat: Add Interactive First Project Tutorial with Step Validation#5126
feat: Add Interactive First Project Tutorial with Step Validation#5126Ashutoshx7 wants to merge 1 commit intosugarlabs:masterfrom
Conversation
|
✅ All Jest tests passed! This PR is ready to merge. |
2 similar comments
|
✅ All Jest tests passed! This PR is ready to merge. |
|
✅ All Jest tests passed! This PR is ready to merge. |
|
In our last meeting, we discussed the possibility of adding a guided tutorial. I’d really appreciate your feedback on this approach and whether this direction aligns with what we had in mind. Thanks! |
|
I think the idea of this. Maybe we can get broader feedback from @pikurasa and others. I did encounter a few issues, such as the palette closing then I clicked next in the tutorial (after I opened the Rhythm palette). And if froze on Card 4. If we do do this, maybe it should appear sooner in the help card sequence. |
|
Hey @Ashutoshx7, this interactive tutorial is looking great! 🎵 I pulled the branch locally to dig into the issues @walterbender mentioned and found a couple of things that might help fix them:
Hope this helps get it merged! Happy to test again whenever you update. 🚀 |
b71a993 to
fa52389
Compare
|
✅ All Jest tests passed! This PR is ready to merge. |
|
✅ All Jest tests passed! This PR is ready to merge. |
|
@walterbender Screen.Recording.2026-02-01.041809.mp4this pr is ready for review |
|
Generally looks good. But maybe @pikurasa and I can sit down with you to go over the details of the script. For example, should we impose a starting condition? Maybe the goal should include an action block? |
|
✅ All Jest tests passed! This PR is ready to merge. |
|
@Ashutoshx7 ESLint failing.. and also resolve merge conflicts. |
|
✅ All Jest tests passed! This PR is ready to merge. |
47cb69f to
ddc067f
Compare
|
❌ Some Jest tests failed. Please check the logs and fix the issues before merging. Failed Tests: |
ddc067f to
65a26bb
Compare
|
❌ Some Jest tests failed. Please check the logs and fix the issues before merging. Failed Tests: |
65a26bb to
9ed7fe1
Compare
|
✅ All Jest tests passed! This PR is ready to merge. |
9ed7fe1 to
4d5b3a7
Compare
|
✅ All Jest tests passed! This PR is ready to merge. |
- Interactive step-by-step tutorial guiding users through creating their first project - 22 guided steps covering start block, rhythm, pitch, flow, tone palettes - Step validation with visual feedback (green checkmarks, purple Next button) - allowInteraction: true on palette steps so sidebar stays accessible - Tutorial card at position 4/38 for early discoverability - Congratulations! card restored as final tour card (38/38) - Remove unused __makeNewNote method (~96 lines dead code)
4d5b3a7 to
b065acd
Compare
|
This PR has merge conflicts with Please rebase your branch: # Add upstream remote (one-time setup)
git remote add upstream https://github.com/sugarlabs/musicblocks.git
# Fetch latest master and rebase
git fetch upstream
git rebase upstream/master
# Resolve any conflicts, then:
git push --force-with-lease origin YOUR_BRANCH
|

Summary
This PR introduces an interactive step-by-step tutorial that guides new users through creating their first Music Blocks project. Unlike the existing static help tour where users simply click through slides, this tutorial validates user actions and only allows progression when each step is completed.
Problem
New users often struggle with Music Blocks' learning curve:
Solution
An interactive tutorial system featuring:
Tutorial Steps
Changes
New Files
Modified Files
index.html- Added script referenceTechnical Highlights
Testing
How to Test
cursorful-video-1768044942838.mp4