-
-
Notifications
You must be signed in to change notification settings - Fork 806
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
Align corner brackets to the top and bottom when scaling #4200
Conversation
Could you provide some images comparing the old and the new behavior? Should make it easier to tell if that's a problem or not (e.g. if the previous behavior was unintended). |
I think there are images in #4188 but i wonder if it's the correct place to apply "fix". And i'm also surprised to see non of other tests fail |
Presumably because these are some of the few delimiters that aren't already vertically centered in most fonts. In fact, looking at http://mirrors.ctan.org/macros/unicodetex/latex/unicode-math/unimath-symbols.pdf they seem to be the only ones. I don't know the motivation behind centering delimiters, but I guess the safest option would be to only disable centering for these specific ones @bluebear94 ? |
This was done purposely, so the issue here is that these specific corner delimiters have to be handled in a different way. Perhaps check how LuaTeX/XeTeX deals with this? The same issue presumably appears there. |
Both LuaTeX and XeTeX have incorrect results with \usepackage{unicode-math}
\[\ulcorner c \urcorner \ulcorner \frac{a}{b} \urcorner\]
\[\left\ulcorner c \right\urcorner \left\ulcorner \frac{a}{b} \right\urcorner\] gives the following in LuaLaTeX: and the following in XeLaTeX: |
Okay, so I guess we'll have to come up with a solution ourselves. LuaTeX at least doesn't seem to center it even with \left and \right, even if they're not scaled at all @bluebear94. Unlike XeTeX, which seems to suffer from the same problem as typst. |
Can these specific delimiters be top and bottom aligned? |
This broke scaling of the corner brackets, U+231C – U+231F. Hopefully this didn’t happen to be needed? Right?
ba05ce5
to
d7931d6
Compare
Thanks! |
I guess linked issue could be closed |
See #4188
This broke scaling of the corner brackets, U+231C – U+231F. Hopefully this didn’t happen to be needed? Right?