Remove space from 'alt=' tag value.#18
Remove space from 'alt=' tag value.#18rtonsing wants to merge 1 commit intoDistributedProofreaders:mainfrom
Conversation
|
🤣 just came here to do exactly the same |
| fs.writeFileSync(fileName, svgCode); | ||
|
|
||
| let alt = `alt=" "`; | ||
| let alt = `alt=""`; |
There was a problem hiding this comment.
you'd do better in many cases to put the tex in alt!
There was a problem hiding this comment.
and if the tex is a single character, it's really bad practice to replace it with a image!
There was a problem hiding this comment.
you'd do better in many cases to put the tex in alt!
But it might not be easy to determine which are the "many cases" (though I agree a single easily pronounceable character would be one such case), and it would be a bit inconsistent to have some of them with the TeX in the alt, and some of them not.
There was a problem hiding this comment.
Do you have an example where tex in alt would be worse than '' in alt?
There was a problem hiding this comment.
but aside from accessibility, using generated images in place of inline math was what I did 29 years ago. And I certainly didn't render inline "A + Bx" as an image!!
There was a problem hiding this comment.
legacy formats (EPUB2, MOBI) probably won't validate but they're pretty awful anyway. I don't think the HTML5 or EPUB3 should have any problems, but if they do, it shouldn't be hard to fix.
There was a problem hiding this comment.
if ppmath is used, images could be embedded for ebookmaker to use in epub2, delete for modern formats
There was a problem hiding this comment.
If you have a sample with mathml for me, I can see if it's trivial or hard.
There was a problem hiding this comment.
Sorry, I.ve only recently become aware of this because I hadn't set up the notifications.
@eshellman I think it is important to use an image ( or other math component) even for single letters because the html font is not under our control so could look quite different to the same letter in a math expression. The arxiv looks interesting.
There was a problem hiding this comment.
@70ray If the typeface is really that important, you have a very good case for font embedding -
4 participants
I suspect it may have been to avoid ebookmaker warnings, but this is undesirable, it ignores a valid issue.