Skip to content
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

Remove locked-figures-aria flag #2064

Merged
merged 5 commits into from
Jan 7, 2025
Merged

Remove locked-figures-aria flag #2064

merged 5 commits into from
Jan 7, 2025

Conversation

nishasy
Copy link
Contributor

@nishasy nishasy commented Dec 28, 2024

Summary:

Remove the locked-figures-aria flag from the Perseus repo
now that the locked figures' aria labels have been out for
a while now.

Issue: https://khanacademy.atlassian.net/browse/LEMS-2274

Test plan:

yarn jest

@nishasy nishasy self-assigned this Dec 28, 2024
Copy link
Contributor

github-actions bot commented Dec 28, 2024

npm Snapshot: Published

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

Example:

yarn add @khanacademy/perseus@PR2064

If you are working in Khan Academy's webapp, you can run:

./dev/tools/bump_perseus_version.sh -t PR2064

Copy link
Contributor

github-actions bot commented Dec 28, 2024

Size Change: -131 B (-0.01%)

Total Size: 1.28 MB

Filename Size Change
packages/perseus-editor/dist/es/index.js 688 kB -34 B (0%)
packages/perseus/dist/es/index.js 419 kB -97 B (-0.02%)
ℹ️ View Unchanged
Filename Size
packages/kas/dist/es/index.js 39 kB
packages/keypad-context/dist/es/index.js 760 B
packages/kmath/dist/es/index.js 4.27 kB
packages/math-input/dist/es/index.js 78 kB
packages/math-input/dist/es/strings.js 1.79 kB
packages/perseus-core/dist/es/index.js 1.48 kB
packages/perseus-linter/dist/es/index.js 22.2 kB
packages/perseus/dist/es/strings.js 5.03 kB
packages/pure-markdown/dist/es/index.js 3.67 kB
packages/simple-markdown/dist/es/index.js 12.5 kB

compressed-size-action

@nishasy nishasy marked this pull request as ready for review December 28, 2024 01:07
@nishasy nishasy requested a review from a team December 28, 2024 01:07
@anakaren-rojas
Copy link
Contributor

remove-locked-figure-flags will be the catchall branch for all your removals?

Copy link
Member

@catandthemachines catandthemachines 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 remove-locked-figure-flags to main January 7, 2025 19:00
@nishasy
Copy link
Contributor Author

nishasy commented Jan 7, 2025

The parent pull-request (#2063) has been merged into main, but this branch (remove-locked-figure-flags-2) now has conflicts with the new base branch. These conflicts must be resolved before checks can complete on this pull-request.

@nishasy nishasy merged commit 55b4615 into main Jan 7, 2025
8 checks passed
@nishasy nishasy deleted the remove-locked-figure-flags-2 branch January 7, 2025 19:06
SonicScrewdriver added a commit that referenced this pull request Jan 8, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @khanacademy/perseus@49.2.1

### Patch Changes

- [#2064](#2064)
[`55b4615d3`](55b4615)
Thanks [@nishasy](https://github.com/nishasy)! - Remove the
locked-figures-aria flag


- [#2063](#2063)
[`85a5b5e44`](85a5b5e)
Thanks [@nishasy](https://github.com/nishasy)! - Remove the
interactive-graph-locked-features-labels flag


- [#2078](#2078)
[`781cc7df6`](781cc7d)
Thanks [@SonicScrewdriver](https://github.com/SonicScrewdriver)! -
Temporarily fixing pi-based strings for Numeric Input


- [#2065](#2065)
[`eefcf5c5c`](eefcf5c)
Thanks [@nishasy](https://github.com/nishasy)! - Remove the
locked-[figureName]-labels flags


- [#2068](#2068)
[`265a93104`](265a931)
Thanks [@benchristel](https://github.com/benchristel)! - Internal:
Redesign discriminated union type parser to have a simpler and more
intuitive interface.


- [#2073](#2073)
[`4bf4960d4`](4bf4960)
Thanks [@benchristel](https://github.com/benchristel)! - Internal:
improve Perseus JSON parsers so they can handle all English-language
exercises


- [#2080](#2080)
[`c9a28b34c`](c9a28b3)
Thanks [@mark-fitzgerald](https://github.com/mark-fitzgerald)! -
[Dropdown] Bugfix - Text in dropdown not in correct vertical position


- [#2073](#2073)
[`4bf4960d4`](4bf4960)
Thanks [@benchristel](https://github.com/benchristel)! - Internal:
improve the error messages produced by the versionedWidgetOptions parser

## @khanacademy/perseus-editor@17.1.1

### Patch Changes

- [#2064](#2064)
[`55b4615d3`](55b4615)
Thanks [@nishasy](https://github.com/nishasy)! - Remove the
locked-figures-aria flag


- [#2063](#2063)
[`85a5b5e44`](85a5b5e)
Thanks [@nishasy](https://github.com/nishasy)! - Remove the
interactive-graph-locked-features-labels flag


- [#2065](#2065)
[`eefcf5c5c`](eefcf5c)
Thanks [@nishasy](https://github.com/nishasy)! - Remove the
locked-[figureName]-labels flags

- Updated dependencies
\[[`55b4615d3`](55b4615),
[`85a5b5e44`](85a5b5e),
[`781cc7df6`](781cc7d),
[`eefcf5c5c`](eefcf5c),
[`265a93104`](265a931),
[`4bf4960d4`](4bf4960),
[`c9a28b34c`](c9a28b3),
[`4bf4960d4`](4bf4960)]:
    -   @khanacademy/perseus@49.2.1
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.

3 participants