Skip to content

[Interactive Graph] Planning document: interactive, unscored IG#3463

Open
handeyeco wants to merge 4 commits intomainfrom
LEMS-3958/planning-not-scored
Open

[Interactive Graph] Planning document: interactive, unscored IG#3463
handeyeco wants to merge 4 commits intomainfrom
LEMS-3958/planning-not-scored

Conversation

@handeyeco
Copy link
Copy Markdown
Contributor

@handeyeco handeyeco commented Apr 8, 2026

Summary:

I took the research from the ticket and used Claude to make a planning document for upcoming work.

Issue: LEMS-3958

@handeyeco handeyeco self-assigned this Apr 8, 2026
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

🗄️ Schema Change: No Changes ✅

@handeyeco handeyeco changed the title Add planning document for unscored IG [Interactive Graph] Planning document: interactive, unscored IG Apr 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

🛠️ Item Splitting: No Changes ✅

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

Size Change: 0 B

Total Size: 499 kB

ℹ️ View Unchanged
Filename Size
packages/kas/dist/es/index.js 20.5 kB
packages/keypad-context/dist/es/index.js 1 kB
packages/kmath/dist/es/index.js 6.36 kB
packages/math-input/dist/es/index.js 98.5 kB
packages/math-input/dist/es/strings.js 1.61 kB
packages/perseus-core/dist/es/index.item-splitting.js 11.9 kB
packages/perseus-core/dist/es/index.js 25.1 kB
packages/perseus-editor/dist/es/index.js 102 kB
packages/perseus-linter/dist/es/index.js 9.3 kB
packages/perseus-score/dist/es/index.js 9.7 kB
packages/perseus-utils/dist/es/index.js 403 B
packages/perseus/dist/es/index.js 196 kB
packages/perseus/dist/es/strings.js 8.27 kB
packages/pure-markdown/dist/es/index.js 1.39 kB
packages/simple-markdown/dist/es/index.js 6.71 kB

compressed-size-action

Comment thread packages/perseus/src/widgets/interactive-graphs/__docs__/notes/not-graded.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

npm Snapshot: Published

Good news!! We've packaged up the latest commit from this PR (42d05a1) and published it to npm. You
can install it using the tag PR3463.

Example:

pnpm add @khanacademy/perseus@PR3463

If you are working in Khan Academy's frontend, you can run the below command.

./dev/tools/bump_perseus_version.ts -t PR3463

If you are working in Khan Academy's webapp, you can run the below command.

./dev/tools/bump_perseus_version.js -t PR3463

Copy link
Copy Markdown
Contributor

@ivyolamit ivyolamit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@handeyeco overall looking good, i just have 2 non-blocking comments.

## Objectives

1. **Editor toggle** — Content creators get a "Graded" switch in the IG widget editor, defaulting to on (graded). The switch is only shown for widgets that opt in.
2. **Hide answer fields when ungraded** — When `graded` is off, the answer-configuration sections in the IG editor are hidden. The stored answer data is preserved so toggling back on restores it.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this include the Start coordinates? i imagine this is only referring to the "Correct answer" section. I wonder if we would still allow Locked figures here?


### 9. Tests and stories

Each change above needs coverage:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we would also need the regression story for this especially it will look different from normal IG

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants