Bug 204188 – [Cocoa] Add ui-sans-serif as a sibling to ui-serif, ui-monospace, and ui-rounded

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Myles C. Maxfield]

[Cocoa] Add ui-sans-serif as a sibling to ui-serif, ui-monospace, and ui-rounded

[Myles C. Maxfield]

Created attachment 383536 [details]
Patch

[Myles C. Maxfield]

Created attachment 383563 [details]
Patch

[Myles C. Maxfield]

Created attachment 383564 [details]
Patch

[Myles C. Maxfield]

The test failures look unrelated.

[Simon Fraser (smfr)]

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?

[Myles C. Maxfield]

(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?" 🤔

[Myles C. Maxfield]

(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.

[WebKit Commit Bot]

Comment on attachment 383564 [details]
Patch

Clearing flags on attachment: 383564

Committed r252710: <https://trac.webkit.org/changeset/252710>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/57373054>

[Myles C. Maxfield]

(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