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

Improve position of stretchy accents. #1735 #1787

Merged
merged 1 commit into from Jul 10, 2017

Conversation

Projects
None yet
2 participants
@dpvc
Copy link
Member

commented Jul 7, 2017

Only bubble up italic correction or skew for nodes with only one child.

Resolves issue #1735

@dpvc dpvc modified the milestone: MathJax v2.7.2 Jul 7, 2017

@zorkow

This comment has been minimized.

Copy link
Member

commented Jul 10, 2017

I've been playing around with the example from the issue and observe some very bizarre behaviour:

  • in HTML-CSS no accent is displayed
    see here
  • in CommonHTML the enriched math becomes huge
    see here and active explorer
  • in SVG it looks good

Maybe we can discuss this tomorrow.

@zorkow
Copy link
Member

left a comment

See my general comment.

@dpvc

This comment has been minimized.

Copy link
Member Author

commented Jul 10, 2017

Both issues seem to be related to the mathsize="36pt" specification, and have nothing really to do with the issue being addressed here. I think they should be added as a new issue instead of holding up this one. (Note that this issue is about horizontal spacing within CHTML only.)

@dpvc

This comment has been minimized.

Copy link
Member Author

commented Jul 10, 2017

The HTML-CSS issue is due to the way HTML-CSS positions the accents, which fails for large mathsize settings. I've opened #1798 for this.

@zorkow

This comment has been minimized.

Copy link
Member

commented Jul 10, 2017

Sounds good.

@zorkow

zorkow approved these changes Jul 10, 2017

@zorkow

This comment has been minimized.

Copy link
Member

commented Jul 10, 2017

CommonHTML problem could be an maction issue. It's independent of accents. I'll file an issue.

@dpvc

This comment has been minimized.

Copy link
Member Author

commented Jul 10, 2017

I'm working on it already. It is more than just mactions, it is the collapsible extension itself, somehow. Just starting to look.

@zorkow

This comment has been minimized.

Copy link
Member

commented Jul 10, 2017

I've filed it in #1800. Feel free to close or move to a11y.

@dpvc

This comment has been minimized.

Copy link
Member Author

commented Jul 10, 2017

Thanks. I'm working on it and will figure out if it is a CHTML or an a11y-based problem (or both). It does seem to only affect CHTML, but that might be just a difference in how CHTML handles mathsize.

@dpvc dpvc merged commit 122c427 into mathjax:develop Jul 10, 2017

@dpvc dpvc deleted the dpvc:issue1735 branch Jul 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.