Skip to content

fix: increase mathjax contextmenu zindex #111

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

Merged
merged 2 commits into from
May 9, 2025

Conversation

ChristopherChudzicki
Copy link
Collaborator

@ChristopherChudzicki ChristopherChudzicki commented May 9, 2025

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/7299

Description (What does it do?)

Increases zindex of mathjax context menu in RemoteTutorDrawer.

Screenshots (if appropriate):

Screenshot 2025-05-09 at 11 36 07 AM

How can this be tested?

  1. yarn bundle-preview
  2. Visit http://localhost:3000/bundle-demo, click "Set Problem Data", "Open Drawer".
  3. Say Hello $E=mc^2$ or something with math.
  4. Try right clicking on math. The context menu should be visible. (ish...it goes off the screen if math is too close right edge, but that's a mathjax issue).

@ChristopherChudzicki ChristopherChudzicki changed the title increase mathjax contextmenu zindex fix: increase mathjax contextmenu zindex May 9, 2025
@ChristopherChudzicki ChristopherChudzicki force-pushed the cc/mathjax-context-zindex branch from 0628f10 to fcd1e24 Compare May 9, 2025 15:37
@ChristopherChudzicki ChristopherChudzicki force-pushed the cc/mathjax-context-zindex branch from fcd1e24 to faa1f1c Compare May 9, 2025 15:53
Copy link
Contributor

@jonkafton jonkafton left a comment

Choose a reason for hiding this comment

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

It's not pretty, but given the constraints - works as intended.

@ChristopherChudzicki ChristopherChudzicki merged commit 0025a63 into main May 9, 2025
11 checks passed
Copy link

github-actions bot commented May 9, 2025

🎉 This PR is included in version 6.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants