Opened 2 years ago

Closed 2 years ago

#34592 closed Cleanup/optimization (fixed)

admin css font does not need explicit apple fonts

Reported by: minusf Owned by: minusf
Component: contrib.admin Version: dev
Severity: Normal Keywords: css, font
Cc: Tom Carrick Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: yes

Description

​https://caniuse.com/?search=system-ui

system-ui is supported now everywhere and -apple-system, BlinkMacSystemFont is not required anymore.

i can make a github PR but the diff is trivial and attaching it here

Attachments (1)

admin_css.diff​ (1.5 KB ) - added by minusf 2 years ago.

Download all attachments as: .zip

Change History (7)

by minusf, 2 years ago

Attachment: admin_css.diff​ added

comment:1 by Natalia Bidart, 2 years ago

Triage Stage: Unreviewed → Accepted
Type: Uncategorized → Cleanup/optimization

Tentatively accepting this ticket based on ​this link from mozilla docs and ​this other, less official, link.

The patch should ideally be a PR, minusf any chance you could do that?

comment:2 by Mariusz Felisiak, 2 years ago

Cc: Tom Carrick added

comment:4 by Mariusz Felisiak, 2 years ago

Owner: changed from nobody to minusf
Status: new → assigned

comment:5 by Mariusz Felisiak, 2 years ago

Triage Stage: Accepted → Ready for checkin

comment:6 by GitHub <noreply@…>, 2 years ago

Resolution: → fixed
Status: assigned → closed

In 8b053c1e:

Fixed #34592 -- Removed Safari/Chrome workaround for system fonts.

Note: See TracTickets for help on using tickets.
Back to Top