-
Notifications
You must be signed in to change notification settings - Fork 14
lspace/rspace have confusing names #232
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
Comments
"leading" and "rear" could be a good pair of words to map to the l/r letters. |
Leading and tRailing are how these are described in mathml 3 and 4, I don't think we should try to introduce "rear" at this stage. But I'd agree with adding a note, the interpretation of these attributes in an RTL context is currently clearer in mathml full which says
|
I agree we could add clarity here - tbh I personally even find it confusing that there is not a link that really defines them exactly - in any case, @davidcarlisle can you add such a note? |
@bkardell yes I'll make a simple PR to add this thanks for the ping. |
Per the WG meeting today: It is too late to change the names, but we agree to the suggestion above and @davidcarlisle will make the simple PR and then we'll consider this resolved for now. |
The I18n WG agrees that a note solves the issue. I believe the note has not actually been added yet at this point, but I'll remove the i18n label in anticipation. |
@davidcarlisle Thanks. Can you please open a PR if that's not already done? I can't find any at https://github.com/w3c/mathml-core/pulls |
(This is part of the I18n WG review.)
3.2.4 Operator, Fence, Separator or Accent
https://www.w3.org/TR/mathml-core/#operator-fence-separator-or-accent-mo
From the algorithm in 3.3.1.2 Layout of it follows that the lspace and rspace attributes of an element add space before, respectively after the element. In a right-to-left formula, that means lspace is on the right and rspace on the left. The names suggest otherwise and when the attributes were created, some 28 years ago, they actually did mean left and right.
To avoid confusion, maybe the spec should explicitly say, in a note, that ‘l’ does not mean ‘left’. (There is an image, figure 10 that contains the words ‘leading’ and ‘trailing’, but doesn't explicitly link them to lspace and rspace.)
The text was updated successfully, but these errors were encountered: