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

Scaling doesn't apply to characters in fallback text font #826

Closed
Herst opened this issue May 23, 2014 · 12 comments

Comments

Projects
None yet
3 participants
@Herst
Copy link

commented May 23, 2014

Let's say you have something like "\text{Gauß}" in your TeX input and you are rendering it in the HTML-CSS format output using the TeX font, then the "ß" is going to be rendered in a fallback font but the "scaling all math" setting is apparently not respected for that character.

From the DOM I see that the "ß" gets puts into its own span but why is the font-size explicitly reduced there? (e.g. to 40% in the case of a global scale factor of 200% on my test page)

Reproducible with the 2.3 and 2.4 beta in different browsers.

DOM snippet:

<span style="font-family: MathJax_Main;" id="MathJax-Span-6" class="mtext">Gau<span style="font-family: STIXGeneral,'Arial Unicode MS',serif; font-size: 40%; font-style: normal; font-weight: normal;">ß</span></span>
@pkra

This comment has been minimized.

Copy link
Member

commented May 26, 2014

Thanks for reporting this. I can reproduce the problem for both TeX and MathML input, HTML-CSS output on FF29, Chrome 34; SVG-output seems to have the reverse problem.

@Herst

This comment has been minimized.

Copy link
Author

commented Aug 4, 2014

JSFiddle of issue: http://jsfiddle.net/Herst/7HEH6/

SVG-output seems to have the reverse problem.

Can't reproduce anymore (in the jsfiddle).

@pkra

This comment has been minimized.

Copy link
Member

commented Aug 4, 2014

I still see this http://jsfiddle.net/7HEH6/5/

@Herst

This comment has been minimized.

Copy link
Author

commented Aug 4, 2014

Oh, you're right, but oddly enough it doesn't work on my jsfiddle page if I change the renderer using the menu (but then the scaling and about window are broken as well).

@pkra

This comment has been minimized.

Copy link
Member

commented Aug 4, 2014

Hm. Did you change the configuration as well? Your configuration only scales the HTML-CSS output.

@Herst

This comment has been minimized.

Copy link
Author

commented Aug 4, 2014

Oh, you're right, forgot about that.

@dpvc dpvc added this to the Next Release milestone Aug 19, 2014

dpvc pushed a commit to dpvc/MathJax that referenced this issue Aug 20, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Aug 20, 2014

The issue826 branch of my fork or MathJax includes a fix.

@dpvc dpvc added Accepted and removed Investigate labels Aug 20, 2014

dpvc pushed a commit to dpvc/MathJax that referenced this issue Aug 21, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Aug 21, 2014

==> Merged.

@dpvc dpvc closed this Aug 21, 2014

@dpvc dpvc added Merged and removed Ready for Review labels Aug 21, 2014

@Herst

This comment has been minimized.

Copy link
Author

commented Aug 24, 2014

Great! I looked at the patch and I saw that the fix concerning the HTML-CSS renderer was just about replacing

100/HTMLCSS.scale

with

HTMLCSS.config.scale/HTMLCSS.scale

in output/HTML-CSS/jax.js.

Though I can still find former in some other parts of jax.js, not sure what they do (for special MathML input?) but just letting you know in case.

dpvc pushed a commit to dpvc/MathJax that referenced this issue Aug 24, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Aug 24, 2014

@Herst, you are absolutely right, there were additional places that also needed fixing. I only took care of the unknown characters (ones that aren't in the MathJax fonts, like the example you gave), but I also needed to change the scaling for text when mtextFontInherit is in effect. I have made the additional patch.

@dpvc dpvc reopened this Aug 24, 2014

@dpvc dpvc added Ready for Review and removed Merged labels Aug 24, 2014

dpvc pushed a commit to dpvc/MathJax that referenced this issue Sep 1, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Sep 1, 2014

==> Merged (again).

@dpvc dpvc closed this Sep 1, 2014

@dpvc dpvc added Merged and removed Ready for Review labels Sep 1, 2014

dpvc pushed a commit to mathjax/MathJax-test that referenced this issue Jan 17, 2015

@dpvc dpvc added Test Available and removed Test Needed labels Jan 17, 2015

@dpvc

This comment has been minimized.

Copy link
Member

commented Jan 17, 2015

==> In Testsuite

MathMLToDisplay/Characters/issue826-1.html
MathMLToDisplay/Characters/issue826-2.html

@dpvc dpvc added Fixed v2.5 and removed Merged labels Jan 30, 2015

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.