[Cocoa] Add ui-sans-serif as a sibling to ui-serif, ui-monospace, and ui-rounded
Created attachment 383536 [details] Patch
Created attachment 383563 [details] Patch
Created attachment 383564 [details] Patch
The test failures look unrelated.
Comment on attachment 383564 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383564&action=review > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1272 > Optional<SystemFontDatabaseCoreText::ClientUse> designSystemUI; These words don't tell me anything. "client use"? "design system UI" - is that a job title?
(In reply to Simon Fraser (smfr) from comment #5) > Comment on attachment 383564 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=383564&action=review > > > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1272 > > Optional<SystemFontDatabaseCoreText::ClientUse> designSystemUI; > > These words don't tell me anything. "client use"? "design system UI" - is > that a job title? "system design" is a term of art: https://developer.apple.com/documentation/appkit/nsfontdescriptorsystemdesign?language=objc I suppose I could rename "design system ui" to "system design." I guess I could rename "client use" to "use case." Or maybe "purpose?" Or maybe "family?" 🤔
(In reply to Myles C. Maxfield from comment #6) > (In reply to Simon Fraser (smfr) from comment #5) > > Comment on attachment 383564 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=383564&action=review > > > > > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1272 > > > Optional<SystemFontDatabaseCoreText::ClientUse> designSystemUI; > > > > These words don't tell me anything. "client use"? "design system UI" - is > > that a job title? > > "system design" is a term of art: > https://developer.apple.com/documentation/appkit/ > nsfontdescriptorsystemdesign?language=objc > > I suppose I could rename "design system ui" to "system design." > > I guess I could rename "client use" to "use case." Or maybe "purpose?" Or > maybe "family?" 🤔 These renames will end up being a bigger patch than this patch here, so I'll do that in a follow-up patch.
Comment on attachment 383564 [details] Patch Clearing flags on attachment: 383564 Committed r252710: <https://trac.webkit.org/changeset/252710>
All reviewed patches have been landed. Closing bug.
<rdar://problem/57373054>
(In reply to Myles C. Maxfield from comment #7) > (In reply to Myles C. Maxfield from comment #6) > > (In reply to Simon Fraser (smfr) from comment #5) > > > Comment on attachment 383564 [details] > > > Patch > > > > > > View in context: > > > https://bugs.webkit.org/attachment.cgi?id=383564&action=review > > > > > > > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1272 > > > > Optional<SystemFontDatabaseCoreText::ClientUse> designSystemUI; > > > > > > These words don't tell me anything. "client use"? "design system UI" - is > > > that a job title? > > > > "system design" is a term of art: > > https://developer.apple.com/documentation/appkit/ > > nsfontdescriptorsystemdesign?language=objc > > > > I suppose I could rename "design system ui" to "system design." > > > > I guess I could rename "client use" to "use case." Or maybe "purpose?" Or > > maybe "family?" 🤔 > > These renames will end up being a bigger patch than this patch here, so I'll > do that in a follow-up patch. https://bugs.webkit.org/show_bug.cgi?id=204486