Page MenuHomePhabricator

Add HiDPI favicons
ClosedPublic

Authored by chad on Nov 6 2014, 8:54 PM.
Tags
None
Referenced Files
F19813907: D10796.diff
Wed, Mar 4, 11:33 AM
F18869199: D10796.diff
Nov 4 2025, 10:23 AM
F18859209: D10796.id25966.diff
Nov 2 2025, 12:31 AM
F18838360: D10796.id25966.diff
Oct 27 2025, 10:23 AM
F18833577: D10796.diff
Oct 26 2025, 4:33 AM
F18749202: D10796.diff
Oct 4 2025, 2:51 AM
F18651237: D10796.diff
Sep 21 2025, 3:37 AM
F18565653: D10796.id.diff
Sep 9 2025, 1:35 PM
Subscribers

Details

Summary

Fixes T5939. Specifically, 16, 32, and 64 bit pngs are in the .ico file. Apple icons are added to support non-Retina and Retina iPad and iPhone. Follows roughly Apple guidelines here: https://developer.apple.com/library/ios/documentation/AppleApplications/Reference/SafariWebContent/ConfiguringWebApplications/ConfiguringWebApplications.html

Test Plan

Tested Chrome, Firefox on Mac. Much cleaner on Retina.

pasted_file (40×91 px, 6 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Add HiDPI favicons.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad edited the test plan for this revision. (Show Details)
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 8 2014, 1:03 AM
This revision was automatically updated to reflect the committed changes.