Skip to content

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

Open
bert-github opened this issue Apr 9, 2024 · 7 comments
Open

lspace/rspace have confusing names #232

bert-github opened this issue Apr 9, 2024 · 7 comments

Comments

@bert-github
Copy link
Contributor

(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.)

@bert-github bert-github added the i18n-needs-resolution Issue the Internationalization Group has raised and looks for a response on. label Apr 9, 2024
@dginev
Copy link

dginev commented Apr 9, 2024

"leading" and "rear" could be a good pair of words to map to the l/r letters.

@davidcarlisle
Copy link
Collaborator

davidcarlisle commented Apr 9, 2024

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

Specifies the leading space appearing before the operator; see 3.2.5.6.4 Spacing around an operator. (Note that before is on the right in a RTL context; see 3.1.5 Directionality.

@bkardell
Copy link
Collaborator

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?

@davidcarlisle
Copy link
Collaborator

@bkardell yes I'll make a simple PR to add this thanks for the ping.

@bkardell
Copy link
Collaborator

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.

@bert-github
Copy link
Contributor Author

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.

@bert-github bert-github removed the i18n-needs-resolution Issue the Internationalization Group has raised and looks for a response on. label Dec 19, 2024
@fred-wang
Copy link
Contributor

yes I'll make a simple PR to add this thanks for the ping.

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants