Bug 263496
| Summary: | Fix Range.getClientRects() to include full grapheme clusters | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | Text | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | darin, fantasai.bugs, karlcow, mmaxfield, rniwa, simon.fraser, webkit-bug-importer, zalan |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
While going through Blink's commit, I came across another test failing in Safari Technology Preview 181 while passing in Chrome Canary 120. NOTE: It also fails in Firefox Nightly 120.
Blink Commit: https://chromium.googlesource.com/chromium/src.git/+/0bac6c9ccc129eaa0a9c209f5f13466fe087ca7c
Test Case (took from 'source.chromium.org'): https://jsfiddle.net/5Lbgt8nf/show
Just raising so we can track it and fix it in future.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/117625675>
Karl Dubost
The tests starts failing for Firefox, Safari for DOMRect.width
Now the question is what is right.
The tests are trying these
testGrapheme,
[
['Latin Combining Diacritical Marks U+301E at start',
'áe',
0,
2],
[
'Latin Combining Diacritical Marks U+301E at mid',
'eáe',
1,
3
],
[
'Latin Combining Diacritical Marks U+301E at end',
'eá',
1,
3
],
[
'Arabic Fatha U+064E at start',
'وَي',
0,
2
],
[
'Arabic Fatha U+064E at mid',
'يوَي',
1,
3
],
[
'Arabic Fatha U+064E at end',
'يوَ',
1,
3
],
]
Actual:
{
"x": 0,
"y": 20.800003051757812,
"width": 0,
"height": 16,
"top": 20.800003051757812,
"right": 0,
"bottom": 36.80000305175781,
"left": 0
}
Expected:
{
"x": 0,
"y": 20.800003051757812,
"width": 7.100006103515625,
"height": 16,
"top": 20.800003051757812,
"right": 7.100006103515625,
"bottom": 36.80000305175781,
"left": 0
}
Probably there are missing WPT tests for this.