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

2.6.0-beta.2: IE11 loses space around equation #1338

Closed
dpvc opened this issue Dec 17, 2015 · 4 comments
Closed

2.6.0-beta.2: IE11 loses space around equation #1338

dpvc opened this issue Dec 17, 2015 · 4 comments
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Available v2.6
Milestone

Comments

@dpvc
Copy link
Member

dpvc commented Dec 17, 2015

As reported at math.stackexchange, the expressions in IE11 are missing some space around them.

This is due to this line. It appears that there are two things involved: (1) the border-spacing:3px isn't having any effect (so the padding:-3px is reducing the space uncompensated by the boarder), and (2) the body element has the focus initially, which triggers this CSS rule.

@dpvc dpvc added the Accepted Issue has been reproduced by MathJax team label Dec 17, 2015
@dpvc
Copy link
Member Author

dpvc commented Dec 17, 2015

I think the main solution is to remove the border-spacing and padding statements and go back to the tight bounding box.

But I still don't like the rule being triggered by the body element being focused. The *:focus .MathJax rule was to handle the problem with links that contain math not having the right outline. So perhaps a rule triggered by *[outline] .MathJax would do it.

@dpvc dpvc added this to the MathJax v2.6 milestone Dec 17, 2015
@dpvc
Copy link
Member Author

dpvc commented Dec 17, 2015

OK, *[outline] .MathJax doesn't work, but body :focus .MathJax seems to work better (this only triggers if a child of the body element is focused, not the body itself).

A similar change needs to be made to CommonHTML as well.

@dpvc
Copy link
Member Author

dpvc commented Dec 18, 2015

The issue1338 branch of my fork of MathJax includes a fix for this.

@dpvc
Copy link
Member Author

dpvc commented Dec 18, 2015

=> Merged.

@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Dec 18, 2015
@dpvc dpvc closed this as completed Dec 18, 2015
@dpvc dpvc added v2.6 Fixed and removed Merged Merged into develop branch labels Jan 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Available v2.6
Projects
None yet
Development

No branches or pull requests

1 participant