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

[Dropdown] Bugfix - Text in dropdown not in correct vertical position #2080

Merged

Conversation

mark-fitzgerald
Copy link
Contributor

@mark-fitzgerald mark-fitzgerald commented Jan 7, 2025

Summary

Additional styling evaded the effectiveness of the original fix for this issue. Styling has been adjusted to only affect dropdown display when in an article, and not an exercise. Also, the same styling fix is applied to .perseus-block-math within the dropdown (and only in articles) for the same margin problem when MathJax is involved.

Issue: LEMS-2742

Test plan:

  1. Open an article with a dropdown (i.e. test everything article)
    • Dropdown content should be vertically centered within the dropdown border
  2. Open an article with a dropdown that includes MathJax (i.e. Numeric Input playground - temporarily)
    • Dropdown content should be vertically centered within the dropdown border
  3. Open an exercise with a dropdown (i.e. Basic Input to Numeric Input last question - temporarily)
    • Dropdown content should be vertically centered within the dropdown border

@mark-fitzgerald mark-fitzgerald self-assigned this Jan 7, 2025
Copy link
Contributor

github-actions bot commented Jan 7, 2025

Size Change: 0 B

Total Size: 1.28 MB

ℹ️ 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-editor/dist/es/index.js 688 kB
packages/perseus-linter/dist/es/index.js 22.2 kB
packages/perseus/dist/es/index.js 419 kB
packages/perseus/dist/es/strings.js 5.05 kB
packages/pure-markdown/dist/es/index.js 3.67 kB
packages/simple-markdown/dist/es/index.js 12.5 kB

compressed-size-action

Copy link
Contributor

github-actions bot commented Jan 7, 2025

npm Snapshot: Published

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

Example:

yarn add @khanacademy/perseus@PR2080

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

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

@mark-fitzgerald mark-fitzgerald marked this pull request as ready for review January 7, 2025 23:50
Copy link
Contributor

@SonicScrewdriver SonicScrewdriver left a comment

Choose a reason for hiding this comment

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

Our style nestings are silly! Thank you so much for fixing this!

@mark-fitzgerald mark-fitzgerald merged commit c9a28b3 into main Jan 8, 2025
8 checks passed
@mark-fitzgerald mark-fitzgerald deleted the bugfix/LEMS-2742-dropdown-text-in-articles-shifted-up branch January 8, 2025 00:01
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.

2 participants