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

Regression with mglyph #2298

Open
NSoiffer opened this issue Jun 22, 2018 · 2 comments
Open

Regression with mglyph #2298

NSoiffer opened this issue Jun 22, 2018 · 2 comments
Labels
Milestone

Comments

@NSoiffer
Copy link

Here's the code:

<math  xmlns="http://www.w3.org/1998/Math/MathML">
	<mi>
		 <mglyph fontfamily="my-braid-font" index="2" alt="23braid"/>
	</mi>
	<mspace width="10px"/>
	<mn>
		 <mglyph fontfamily="Curlz MT" index="65" alt="script 'A'"/>
	</mn>
	<mspace width="10px"/>
	<mn>
		 <mglyph fontfamily="Arial" index="97" alt="lowercase 'a'"/>
	</mn>
</math>

Here's V2's rendering:
image

Here's V3's rendering:
image

@dpvc
Copy link
Member

dpvc commented May 31, 2019

The current mglyph implementation is somewhat more restricted than the v2 version. It doesn't currently implement the deprecated fontfamily and index attributes (and may not in the future). Actually, it does fontfamily as that was available on all elements, but it doesn't do the index attribute. So you are getting broken image icons for the missing src attribute, and the content of the alt attributes since the images aren't showing.

I'll look into implementing the index attribute for the future.

@dpvc
Copy link
Member

dpvc commented Jan 17, 2020

I'm transferring this issue to the main MathJax issue tracker, so it is easier to locate.

@dpvc dpvc transferred this issue from mathjax/MathJax-demos-web Jan 17, 2020
@dpvc dpvc changed the title Regression with mglyph [v3] Regression with mglyph Jan 17, 2020
@dpvc dpvc added the v3 label Feb 19, 2020
@dpvc dpvc changed the title [v3] Regression with mglyph Regression with mglyph Feb 24, 2020
@dpvc dpvc self-assigned this May 12, 2020
dpvc added a commit to mathjax/MathJax-src that referenced this issue Feb 11, 2022
dpvc added a commit to mathjax/MathJax-src that referenced this issue Feb 11, 2022
dpvc added a commit to mathjax/MathJax-src that referenced this issue Apr 6, 2022
Add support for mglyph use of fontfamily/index.  (mathjax/MathJax#2298)
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Apr 6, 2022
@dpvc dpvc added this to the 3.3.0 milestone Aug 5, 2022
@dpvc dpvc removed their assignment Aug 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants