[wp-trac] [WordPress Trac] #54370: Add an option to configure the site icon in general settings
WordPress Trac
noreply at wordpress.org
Thu Feb 15 10:13:40 UTC 2024
#54370: Add an option to configure the site icon in general settings
---------------------------------------+-----------------------
Reporter: jameskoster | Owner: jorbin
Type: task (blessed) | Status: reopened
Priority: normal | Milestone: 6.5
Component: General | Version: 5.9
Severity: normal | Resolution:
Keywords: has-screenshots has-patch | Focuses:
---------------------------------------+-----------------------
Comment (by afercia):
> I think removing this was premature and it would be best to add it back
and use it for aria-describedby since that will be consistent with the
rest of the experience on this page.
I'm not sure you are fully getting how `aria-describedby` is meant to be
used. Usually, we use it to associate an input field to a description. In
this case, there's no input field. There's buttons that get dynamically
hidden. Associating the 'choose' button to the descriptions is technically
possible but not recommended, as the button gets hidden when an image is
set. I don't think we should use `aria-describedby` in this UI and thus
the IDs are unnecessary. They weren't used anyways in the original commit.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/54370#comment:85>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list