Skip to content

Conversation

@mark-fitzgerald
Copy link
Contributor

Summary:

This PR converts the Aphrodite styling into standard CSS. However, since this widget uses a Wonder Blocks component and overrides its styling, the legacy Aphrodite code is kept in a separate file to be deleted later (once Wonder Blocks has migrated to CSS Modules).

Issue: LEMS-3818

Test plan:

Regression tests pass

@mark-fitzgerald mark-fitzgerald self-assigned this Dec 30, 2025
@mark-fitzgerald mark-fitzgerald requested a review from a team December 30, 2025 00:34
@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

🗄️ Schema Change: No Changes ✅

@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

🛠️ Item Splitting: No Changes ✅

@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

Size Change: +100 B (+0.02%)

Total Size: 499 kB

Filename Size Change
packages/perseus/dist/es/index.js 200 kB +100 B (+0.05%)
ℹ️ View Unchanged
Filename Size
packages/kas/dist/es/index.js 20.8 kB
packages/keypad-context/dist/es/index.js 1 kB
packages/kmath/dist/es/index.js 5.98 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 13.2 kB
packages/perseus-core/dist/es/index.js 26.1 kB
packages/perseus-editor/dist/es/index.js 97.5 kB
packages/perseus-linter/dist/es/index.js 8.65 kB
packages/perseus-score/dist/es/index.js 9.25 kB
packages/perseus-utils/dist/es/index.js 403 B
packages/perseus/dist/es/strings.js 7.78 kB
packages/pure-markdown/dist/es/index.js 1.39 kB
packages/simple-markdown/dist/es/index.js 6.72 kB

compressed-size-action

@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

npm Snapshot: Published

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

Example:

pnpm add @khanacademy/perseus@PR3163

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

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

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

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

…check for "prefers reduced motion" in CSS and restore it in legacy styling.

Adjust unit tests and snapshots accordingly.
Copy link
Contributor

@anakaren-rojas anakaren-rojas left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from deploy/perseus-january-2026 to main January 8, 2026 20:15
@github-actions github-actions bot added schema-change Attached to PRs when we detect Perseus Schema changes in it item-splitting-change labels Jan 8, 2026
…anch 'main' into LEMS-3818-migrate-aphrodite-to-css-modules-for-explanation
@github-actions github-actions bot removed schema-change Attached to PRs when we detect Perseus Schema changes in it item-splitting-change labels Jan 9, 2026
@mark-fitzgerald mark-fitzgerald merged commit 8a3fcc6 into main Jan 9, 2026
10 of 11 checks passed
@mark-fitzgerald mark-fitzgerald deleted the LEMS-3818-migrate-aphrodite-to-css-modules-for-explanation branch January 9, 2026 18:34
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.

3 participants