[wp-trac] [WordPress Trac] #60716: Font Library: Replace error message "A font face matching those settings already exists" with notice
WordPress Trac
noreply at wordpress.org
Tue Apr 9 12:34:00 UTC 2024
#60716: Font Library: Replace error message "A font face matching those settings
already exists" with notice
-------------------------------------------------+-------------------------
Reporter: oglekler | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: 6.6
Component: Editor | Version: 6.5
Severity: normal | Resolution:
Keywords: has-screenshots needs-patch needs- | Focuses:
design-feedback |
-------------------------------------------------+-------------------------
Comment (by akrocks):
Hi @jorbin,
How about using a [https://developer.wordpress.org/block-editor/reference-
guides/components/notice/ Notice] with `info` as `status`?
Somewhat like (Note: For now we can skip the text appearing in image &
update it later, once the UI is finalised 😇)
[[Image(ticket:60716:Screenshot 2024-04-09 at 5.42.12 PM.png)]]
Also, as we are going to update the Font upload modal & the code change
will be in [https://github.com/WordPress/gutenberg/tree/trunk/packages
/edit-site edit-site] package of [https://github.com/WordPress/gutenberg/
Gutenberg repo], should I go ahead & create an issue on
[https://github.com/WordPress/gutenberg/ Gutenberg repo] for this (if not
already created).
Have already created a draft PR containing error bifurcation (into
error/warning/info) logic. We can further update the logic/UI as per the
discussion/feedbacks. 😉
PR Link: https://github.com/WordPress/gutenberg/pull/60594
--
Ticket URL: <https://core.trac.wordpress.org/ticket/60716#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list