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

Fix vector arrow positioning in CHTML. #1543 #1550

Merged
merged 1 commit into from Aug 5, 2016

Conversation

Projects
None yet
3 participants
@dpvc
Copy link
Member

commented Jul 5, 2016

Change class to mjx-box so that width will have the proper effect (due to fix in CSS in PR #1448). Resolves issue #1543.

@pkra pkra added this to the MathJax v2.x.x milestone Jul 6, 2016

@@ -1882,7 +1882,7 @@
// something, so put them over a space and remove the space's width
//
node = node.firstChild;
var space = CHTML.Element("mjx-span",{style:{width:".25em","margin-left":"-.25em"}});
var space = CHTML.Element("mjx-box",{style:{width:".25em","margin-left":"-.25em"}});

This comment has been minimized.

Copy link
@zorkow

zorkow Jul 23, 2016

Member

I assume this does not have any implications on CSS behaviour?

This comment has been minimized.

Copy link
@dpvc

dpvc Jul 23, 2016

Author Member

Only the desired one of making it work. :-)

The only CSS that these two set is the display value, and that is what needs to be changed. So there are no side-effects of this.

@zorkow

This comment has been minimized.

Copy link
Member

commented Aug 4, 2016

lgtm.

@dpvc dpvc merged commit 71369c3 into mathjax:develop Aug 5, 2016

@dpvc dpvc deleted the dpvc:issue1543 branch Aug 5, 2016

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.