Bug 228018

Summary: [Cocoa] ".SF Arabic" should not be able to be looked up by name (because it is dot-prefixed)
Product: WebKit Reporter: Ebrahim Byagowi <ebrahim>
Component: TextAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: mmaxfield, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Arabic letters aren't connected
none
Patch
none
Patch simon.fraser: review+

Ebrahim Byagowi
Reported 2021-07-16 01:38:05 PDT
Created attachment 433664 [details] Arabic letters aren't connected Steps: 0. Upgrade to macOS Monterey beta. 1. data:text/html;charset=utf8,<b style="font-family:'.SF Arabic'">سیب 2. Use touchpad pinch to zoom to zoom on the text to show Actual: There is a space between Arabic letters Expected: To have all letters connected
Attachments
Arabic letters aren't connected (87.66 KB, image/png)
2021-07-16 01:38 PDT, Ebrahim Byagowi
no flags
Patch (5.97 KB, patch)
2021-07-21 13:44 PDT, Myles C. Maxfield
no flags
Patch (4.23 KB, patch)
2021-07-21 16:53 PDT, Myles C. Maxfield
simon.fraser: review+
Ebrahim Byagowi
Comment 1 2021-07-16 02:03:53 PDT
There is a difference between these two, data:text/html;charset=utf8,<b style="font-family:'.SF Arabic'">سیب data:text/html;charset=utf8,<b style="font-family:system-ui">سیب Their fonts are the same but getting bold in different ways, maybe somehow the disconnect letters is because of this
Radar WebKit Bug Importer
Comment 2 2021-07-16 10:30:44 PDT
Myles C. Maxfield
Comment 3 2021-07-21 13:42:11 PDT
*** Bug 228163 has been marked as a duplicate of this bug. ***
Myles C. Maxfield
Comment 4 2021-07-21 13:44:07 PDT
*** Bug 228163 has been marked as a duplicate of this bug. ***
Myles C. Maxfield
Comment 5 2021-07-21 13:44:12 PDT
Myles C. Maxfield
Comment 6 2021-07-21 16:53:46 PDT
Simon Fraser (smfr)
Comment 7 2021-07-22 11:18:28 PDT
Comment on attachment 433975 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=433975&action=review Is it OK to do this on all OS versions? > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1114 > + || equalLettersIgnoringASCIICase(family, ".sf arabic")) { I would move this up to below ".sf ui mono"
Myles C. Maxfield
Comment 8 2021-07-22 15:40:28 PDT
Myles C. Maxfield
Comment 9 2021-07-22 17:49:23 PDT
(In reply to Simon Fraser (smfr) from comment #7) > Comment on attachment 433975 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=433975&action=review > > Is it OK to do this on all OS versions? SF Arabic is new to Monterey & iOS 15
Note You need to log in before you can comment on or make changes to this bug.