[wp-trac] [WordPress Trac] #16434: Give site admin ability to upload favicon in Settings, General
WordPress Trac
noreply at wordpress.org
Mon Jun 29 02:48:17 UTC 2015
#16434: Give site admin ability to upload favicon in Settings, General
-------------------------------------+-------------------------------------
Reporter: jane | Owner: johnbillion
Type: task (blessed) | Status: accepted
Priority: normal | Milestone: 4.3
Component: Customize | Version: 3.1
Severity: normal | Resolution:
Keywords: ux-feedback needs-patch | Focuses: ui, accessibility,
needs-screenshots | administration
-------------------------------------+-------------------------------------
Comment (by celloexpressions):
In-progress Customizer patch is now on #29211. We're still waiting for the
initial commit for the settings version so that the base API can be reused
with only the administration layer changing. The experience will be
smoother there since cropping is handled directly in the media modal so
you never leave the page/break the flow. If this is planning on being
saved up until right before beta, I would appreciate knowing that sooner
rather than later so that I can just copy the API to build into the
Customizer version so that it can also be ready in time, but that seems
like a silly way to go about this.
Sidenote: we should use a more generic browser chrome image if we really
want to have that there. And we might as well put the site title there too
if it isn't already in the patch. We also know the site url, so the start
of that would be nicer than "www.". In the Customizer, I'm planning on
just live-updating the icon in the browser window via JS, which most
modern browsers support (Slack does this, for example). Seems like that
should be adequate? Would appreciate more screenshots of what the patch
does posted on this ticket though.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/16434#comment:184>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list