Page MenuHomePhabricator

Add HiDPI favicons
ClosedPublic

Authored by chad on Nov 6 2014, 8:54 PM.
Tags
None
Referenced Files
F14033968: D10796.diff
Sat, Nov 9, 9:05 PM
F14022034: D10796.diff
Wed, Nov 6, 2:53 PM
F14011058: D10796.diff
Thu, Oct 31, 4:07 PM
F14010209: D10796.diff
Thu, Oct 31, 6:30 AM
F13971420: D10796.diff
Thu, Oct 17, 1:19 PM
Unknown Object (File)
Oct 9 2024, 6:41 PM
Unknown Object (File)
Oct 9 2024, 2:57 PM
Unknown Object (File)
Oct 9 2024, 11:19 AM
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.