Skip to content

Fix missing tutorial progress badges in My Projects grid view#11439

Open
Copilot wants to merge 2 commits into
masterfrom
copilot/show-tutorial-progress-in-grid-view
Open

Fix missing tutorial progress badges in My Projects grid view#11439
Copilot wants to merge 2 commits into
masterfrom
copilot/show-tutorial-progress-in-grid-view

Conversation

Copilot AI commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Tutorial progress badges (e.g. "3/5", "6/6") shown on project cards in the home screen carousel were not appearing in the ScriptManagerDialog grid view ("View All").

Root cause

The grid view in scriptmanager.tsx rendered ProjectsCodeCard without computing or passing tutorialStep/tutorialLength props. The list view in the same file and the home screen carousel both correctly derived these from header.tutorial and header.tutorialCompleted.

Changes

  • webapp/src/scriptmanager.tsx — Added tutorialStep and tutorialLength computation in the grid view map, matching the existing pattern in the list view, and passed them to ProjectsCodeCard:
const tutorialStep =
    scr.tutorial ? scr.tutorial.tutorialStep
        : scr.tutorialCompleted ? scr.tutorialCompleted.steps - 1
            : undefined;
const tutorialLength =
    scr.tutorial ? scr.tutorial.tutorialStepInfo.length
        : scr.tutorialCompleted ? scr.tutorialCompleted.steps
            : undefined;

return <ProjectsCodeCard
    ...
    tutorialStep={tutorialStep}
    tutorialLength={tutorialLength}
/>

Copilot AI changed the title [WIP] Fix tutorial progress icon missing in grid view Fix missing tutorial progress badges in My Projects grid view Jun 26, 2026
Copilot AI requested a review from abchatra June 26, 2026 23:28
@abchatra abchatra marked this pull request as ready for review June 26, 2026 23:45
@abchatra abchatra requested a review from a team June 26, 2026 23:45
@abchatra

Copy link
Copy Markdown
Collaborator

Fixes microsoft/pxt-microbit#6972

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants