-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
\not a (etc) not italic #2617
Comments
OK, here's what's happening. The The only alternative would be to do two elements, like |
Ah, I should've thought of that (I had actually looked at the MML). I'll check regarding STIX2 (but this was just a tangential find while working on the unicode-math extension). |
I suppose another alternative is to not count combining characters when determining whether the content is one character or not. I'm not sure if that would be against the spec or not. I think it does make sense to consider them as a single glyph, so this might be the best approach. This will become more important when we handle combining characters better during TeX input, for example. It turns out that v2 accidentally doesn't count the U+0338 because of how it was added on, and so renders using italics. |
Sorry for dropping the ball on this one. @davidmjones had looked at unicode-math / xelatex behavior and IIRC that was a bit dodgy but MathJax v2's behavior was good with him. @davidmjones can you comment regarding the italic vs nonitalic slash issue in #2617 (comment)? |
OK, I have tested out the following idea: don't count combining characters as part of the length of an |
@pkra, any thoughts on my solution of dropping the italic U+0338 and not counting combining characters when determining the length of the |
@dpvc I checked with @davidmjones and this sounds good. |
OK, I've pushed the |
Allow 'single-character' mi's to include trailing combining characters (mathjax/MathJax#2617)
E.g.,$\not a$ comes out normal (u0061) whereas $\not{a}$ comes out math italic (u1D44E).
Screencap from https://www.mathjax.org/#demo
The text was updated successfully, but these errors were encountered: