Skip to content

Add overflow: visible to svgs used for brackets and such. #36

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 1 commit into from
Feb 20, 2025

Conversation

drgrice1
Copy link
Member

@drgrice1 drgrice1 commented Feb 5, 2025

At the default zoom level for me in both Firefox and Chrome the bottom of a brace or bracket is clipped. If I increase or decrease the zoom level the clipping stops. It also seems to usually only happen in the first MathQuill input on the page, and not the later ones. Also, this does not happen with the MathQuill test pages. It only happens in webwork2. So this may have something to do with some Bootstrap css.

Here is a screen shot of what I am seeing:
svg-clipping

Adding overflow: visible prevents this. I don't think there is ever a situation where these svgs should be clipped, so I don't think there is any downside to adding this.

Also run npx update-browserslist-db@latest because I am seeing warnings about that.

I am seeing clipping occur on the bottoms of brackets and braces in some
instances.  Adding `overflow: visible` prevents this.  I don't think
there is ever a situation where these svgs should be clipped, so I don't
think there is any downside to adding this.

At the default zoom level for me in both Firefox and Chrome the bottom
of a brace or bracket is clipped.  If I increase or decrease the zoom
level the clipping stops.  It also seems to usually only happen in the
first MathQuill input on the page, and not the later ones.  So overall
very odd.

Also run `npx update-browserslist-db@latest` because I am seeing
warnings about that.
drgrice1 added a commit to drgrice1/pg that referenced this pull request Feb 5, 2025
This corresponds to openwebwork/mathquill#36,
and just upgrades to the now published version with that tweak.
Copy link
Member

@pstaabp pstaabp left a comment

Choose a reason for hiding this comment

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

Not seeing clipping in Safari. Looks fine with the new css as well.

drgrice1 added a commit to drgrice1/pg that referenced this pull request Feb 12, 2025
This corresponds to openwebwork/mathquill#36,
and just upgrades to the now published version with that tweak.
@Alex-Jordan Alex-Jordan merged commit a1b3d63 into openwebwork:main Feb 20, 2025
2 checks passed
@drgrice1 drgrice1 deleted the svg-overflow-visible branch February 20, 2025 10:38
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