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.

Sign up for GitHub

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

Jump to bottom

Define what happens with edge cases of RadicalKernBeforeDegree and RadicalKernAfterDegree #213

Closed
fred-wang opened this issue May 21, 2020 · 2 comments
Closed

Define what happens with edge cases of RadicalKernBeforeDegree and RadicalKernAfterDegree #213

fred-wang opened this issue May 21, 2020 · 2 comments
Labels
css / html5 Issues related to CSS or HTML5 interoperability MathML Core Issues affecting the MathML Core specification opentype / tex Issues related to OpenType or TeX rules

Comments

@fred-wang
Copy link

I'm opening this as it's listed on the spec and not tracked yet here. Basically, the radical kernings are:

  • RadicalKernAfterDegree which is negative and normally small.
  • RadicalKernBeforeDegree which is normally nonnegative.

The horizontal positions of mroot is basically done in that order: 0 ; +kernafter space | +index inline size | +kernbefore space | +base inline size ; inline size.

Now if the font has weird values for kernafter and kernbefore, or even just if index has zero inline size then the (preferred) inline sizes can become negative.
@fred-wang fred-wang added MathML Core Issues affecting the MathML Core specification css / html5 Issues related to CSS or HTML5 interoperability opentype / tex Issues related to OpenType or TeX rules need tests Issues related to writing WPT tests need resolution Issues needing resolution at MathML Refresh CG meeting need specification update Issues requiring specification changes labels May 21, 2020
@fred-wang
Copy link
Author

Proposal: When doing layout, using max(RadicalKernBeforeDegree, 0) and max(RadicalKernAfterDegree, -index inline size).

@NSoiffer NSoiffer mentioned this issue May 22, 2020
Closed
@fred-wang
Copy link
Author

consensus from june 1: approved

fred-wang added a commit to w3c/mathml-core that referenced this issue Jun 2, 2020
@fred-wang
Define what happens with edge cases of RadicalKernBeforeDegree and Ra…
265b20b 
…dicalKernAfterDegree

w3c/mathml#213
@fred-wang fred-wang added level-2 and removed need resolution Issues needing resolution at MathML Refresh CG meeting need specification update Issues requiring specification changes level-2 labels Jun 2, 2020
fred-wang added a commit to web-platform-tests/wpt that referenced this issue Jun 2, 2020
@fred-wang
Add a test to check edge cases of radical kerning.
3ad321f 
w3c/mathml#213
@fred-wang fred-wang mentioned this issue Jun 2, 2020
Merged
fred-wang added a commit to web-platform-tests/wpt that referenced this issue Jun 2, 2020
@fred-wang
Add a test to check edge cases of radical kerning. (#23915)
0546635 
w3c/mathml#213
@fred-wang fred-wang removed the need tests Issues related to writing WPT tests label Jun 2, 2020
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Jun 8, 2020
@fred-wang @moz-wptsync-bot
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
bc02f16 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915
xeonchen pushed a commit to xeonchen/gecko that referenced this issue Jun 8, 2020
@fred-wang @moz-wptsync-bot
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
d7d654f 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Jun 9, 2020
@fred-wang
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
1da0a7c 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778
xeonchen pushed a commit to xeonchen/gecko that referenced this issue Jun 9, 2020
@fred-wang
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
b7fd022 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Jun 10, 2020
@fred-wang
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
d26d3aa 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778
xeonchen pushed a commit to xeonchen/gecko that referenced this issue Jun 10, 2020
@fred-wang
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
bd7ddd8 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Jun 11, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
286c9e8 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

UltraBlame original commit: 48a70f17dca659b1b47fe40a4d0a9a2419fb7174
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Jun 11, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
98ebdb0 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778

UltraBlame original commit: a820bab0bc8cc3226d7f78968157e63d42d788f0
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Jun 11, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
9ab0661 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778

UltraBlame original commit: a79a7fff903f6c7fbc1e7f26bc210bc0abd269bd
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Jun 12, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
e5013c9 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

UltraBlame original commit: 48a70f17dca659b1b47fe40a4d0a9a2419fb7174
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Jun 12, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
e52685f 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778

UltraBlame original commit: a820bab0bc8cc3226d7f78968157e63d42d788f0
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Jun 12, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
59f9b14 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778

UltraBlame original commit: a79a7fff903f6c7fbc1e7f26bc210bc0abd269bd
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Jun 12, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
3fe5e1e 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

UltraBlame original commit: 48a70f17dca659b1b47fe40a4d0a9a2419fb7174
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Jun 12, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
7839ac0 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778

UltraBlame original commit: a820bab0bc8cc3226d7f78968157e63d42d788f0
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Jun 12, 2020
@marco-c
Bug 1642656 [wpt PR 23915] - Add a test to check edge cases of radica…
41d5e81 
…l kerning., a=testonly

Automatic update from web-platform-tests
Add a test to check edge cases of radical kerning. (#23915)

w3c/mathml#213
--

wpt-commits: 0546635a0f399f57b0f26d804926c2ca5049e376
wpt-pr: 23915

Differential Revision: https://phabricator.services.mozilla.com/D78778

UltraBlame original commit: a79a7fff903f6c7fbc1e7f26bc210bc0abd269bd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
css / html5 Issues related to CSS or HTML5 interoperability MathML Core Issues affecting the MathML Core specification opentype / tex Issues related to OpenType or TeX rules
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fred-wang