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

mspace linebreak despite dimension #388

Closed
pkra opened this issue Feb 8, 2013 · 4 comments

Comments

Projects
None yet
3 participants
@pkra
Copy link
Member

commented Feb 8, 2013

IIUC,

<math>
<mn>1</mn><mspace linebreak="newline" width="thinmathspace"></mspace><mn>2</mn>
</math> 

should not produce a linebreak according to MML specs.

@fred-wang

This comment has been minimized.

Copy link
Contributor

commented Feb 11, 2013

Yes, I think you're right.

"Specifies the desirability of a linebreak at this space. This attribute should be ignored if any dimensional attribute is set."

@dpvc

This comment has been minimized.

Copy link
Member

commented Feb 11, 2013

OK, that looks right. The line breaking should be ignored if one of the dimensions is given explicitly (this phrase was not present when I wrote the code to process explicit line breaks — I should probably get a fresh copy of the spec for reference, since there are some changes since I downloaded mine).

That should not be too hard to fix. That would require modifications to the hasNewLine method of the MML.mspace object in the mml element jax jax.js file, and the HTMLbetterBreak and SVGbetterBreak methods added to mspace in the multiline.js files for the HTML-CSS and SVG output jax.

@fred-wang

This comment has been minimized.

Copy link
Contributor

commented Mar 25, 2013

=> In testsuite

@fred-wang

This comment has been minimized.

Copy link
Contributor

commented Mar 25, 2013

MathMLToDisplay/Topics/LineBreak/issue388.html

@ghost ghost assigned fred-wang Mar 25, 2013

dpvc pushed a commit to dpvc/MathJax that referenced this issue Apr 1, 2013

@dpvc dpvc closed this May 17, 2013

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.