Fixing resize behavior for Expression Widget

[catandthemachines]

Summary:

Enabling rootBoundary mode for Popover implementation in Expression Widget. This ensures the popover container does not snap depending on the viewport and instead positions it's contend based on where there is room in the page document. This change is needed an intended to ensure the Expression Widget is accessible to users in 400% or very small screen sizes.

Issue: LEMS-2085

Test plan:

1. Viewing disabled viewport functionality.
   • Go to /?path=/docs/perseus-widgets-expression--docs and open an expression widget keypad.
   • Notice that the keypad does not move to stay in the viewport when scrolling up and down the document.
2. Viewing improved 400% zoom experience
   • Go to /iframe.html?args=&id=perseus-widgets-expression--desktop&viewMode=story
   • Increase screen zoom to 400% or minimize screen size to 320 x 256.
   • Notice that you can open the keypad and it does not move outside the page and you can scroll and reach content within the keypad even if it's larger the they viewport size.

[khan-actions-bot]

Gerald

Required Reviewers

• @Khan/perseus for changes to yarn.lock, .changeset/quiet-news-relate.md, dev/package.json, packages/math-input/package.json, packages/perseus/package.json, packages/perseus-editor/package.json, packages/perseus/src/components/math-input.tsx

---

Don't want to be involved in this pull request? Comment #removeme and we won't notify you of further changes.

[github-actions]

Size Change: +16 B (0%)

Total Size: 860 kB

Filename	Size	Change
packages/perseus/dist/es/index.js	417 kB	+16 B (0%)

ℹ️ View Unchanged

Filename	Size
packages/kas/dist/es/index.js	38.3 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	277 kB
packages/perseus-linter/dist/es/index.js	22.2 kB
packages/perseus/dist/es/strings.js	3.36 kB
packages/pure-markdown/dist/es/index.js	3.67 kB
packages/simple-markdown/dist/es/index.js	12.4 kB

_{compressed-size-action}

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.97%. Comparing base (873f107) to head (f5346d5).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1548      +/-   ##
==========================================
+ Coverage   70.38%   70.97%   +0.58%     
==========================================
  Files         521      524       +3     
  Lines      107315   107349      +34     
  Branches     7796    11652    +3856     
==========================================
+ Hits        75538    76192     +654     
+ Misses      31587    31157     -430     
+ Partials      190        0     -190     

Files	Coverage Δ
packages/perseus/src/components/math-input.tsx	82.06% <100.00%> (+0.21%)	⬆️

... and 146 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 873f107...f5346d5. Read the comment docs.

[github-actions]

npm Snapshot: Published

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

Example:

yarn add @khanacademy/perseus@PR1548

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

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

[handeyeco]

Looks good to me. My only concerns:

• Failing TS check
• I wonder if we need to update all WB/WS at the same time?

[catandthemachines]

Looks good to me. My only concerns:

• Failing TS check
• I wonder if we need to update all WB/WS at the same time?

Yeah the weird failing type check made me think I needed to update all of them at the same time. Will keep digging into why it's mis-behaving, and see if it's needed.

[catandthemachines]

Looks good to me. My only concerns:

• Failing TS check
• I wonder if we need to update all WB/WS at the same time?

Yeah the weird failing type check made me think I needed to update all of them at the same time. Will keep digging into why it's mis-behaving, and see if it's needed.

Okay after wrangling with the typing I can summarize why it's there. The update to tooltip also appears to have included a dependency update to the color object Tooltip accepts. So to fix this I just needed to update the tokens package version. So the only packages updated are tooltip, popover, and tokens. 😊