[wp-trac] [WordPress Trac] #16434: Give site admin ability to upload favicon in Settings, General

[WordPress Trac]

#16434: Give site admin ability to upload favicon in Settings, General
--------------------------+------------------------------------------------
 Reporter:  jane          |       Owner:  obenland
     Type:  task          |      Status:  accepted
  (blessed)               |   Milestone:  4.3
 Priority:  normal        |     Version:  3.1
Component:  Customize     |  Resolution:
 Severity:  normal        |     Focuses:  ui, accessibility, administration
 Keywords:  ux-feedback   |
  has-patch               |
--------------------------+------------------------------------------------

Comment (by ryan):

 Replying to [comment:210 ryan]:
 > Replying to [comment:209 ryan]:
 > > With the 4.3-beta1-33054 nightly, cropping is skipped entirely.
 Click/tap Set as Site Icon in the modal and you are immediately directed
 back to Settings > General. Tested with Macnchrome and iPhone 6+.
 >
 > > Site Icon: Skip cropping if image has the correct size.
 >
 > Ah. That's why? Expected with an image straight from a phone's camera
 roll?

 I was choosing the first image in the gallery view of the media lib to
 test this. The first image happened to be a cropped site icon from a
 previous test. I couldn't tell I was picking the already cropped image on
 iPhone 6+ or Macnchrome.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/16434#comment:211>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform