Problem/Motivation

The default favicon in /misc/favicon.ico is 16x16 and looks awful on retina screens.

Proposed resolution

Replace misc/favicon.ico with a retina version. See https://github.com/audreyr/favicon-cheat-sheet for comprehensive information but in this case we are merely trying to fix the basic icon.

Remaining tasks

  • Create a 32x32 favicon.
  • Test on various devices.
  • Backport.

User interface changes

Non-pixelated favicons should appear on browser tabs.

Before:

After:

API changes

Any derived themes that may somehow depend on a 16x16 icon could have an issue.

Original report by @aendrew

Files: 
CommentFileSizeAuthor
#21 1883058-21.patch6.63 KBcilefen
PASSED: [[SimpleTest]]: [MySQL] 41,232 pass(es). View
#5 after.png43.01 KBcilefen
#5 before.png33.81 KBcilefen
#4 1883058-4.patch6.65 KBcilefen
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 1883058-4.patch. Unable to apply patch. See the log in the details link for more information. View

Comments

aendrew’s picture

Status: Active » Needs review
FileSize
2.62 KB
PASSED: [[SimpleTest]]: [MySQL] 39,658 pass(es). View

And a patch. File used is the one specified above (I.e., http://drupal.org/misc/favicon.ico).

aendrew’s picture

Title: Default Drupal drop icon needs a Retina-quality version » Default Drupal "drop" favicon needs a Retina-quality version

Updating title for clarity.

aendrew’s picture

Issue summary: View changes

Clarified "perfect".

aendrew’s picture

Issue summary: View changes

Added article link.

cilefen’s picture

Version: 7.18 » 8.x-dev
Component: other » Stark theme
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Novice

Updated the summary, tagged "novice", changed to 8.x and component "Stark theme".

cilefen’s picture

Issue summary: View changes
Status: Needs work » Needs review
FileSize
6.65 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 1883058-4.patch. Unable to apply patch. See the log in the details link for more information. View
cilefen’s picture

FileSize
33.81 KB
43.01 KB

Before:

After:

cilefen’s picture

Issue summary: View changes
davidhernandez’s picture

I'm struggling to think why a theme would depend on the 16x16 file. Are there use cases anyone knows of? I believe it is documented that older browsers will size down the larger image.

cilefen’s picture

The icon file in the patch contains 16x16 and 32x32.

cilefen’s picture

Issue tags: +Needs manual testing
cilefen’s picture

Issue summary: View changes
cilefen’s picture

Issue tags: +Needs usability review
Bojhan’s picture

Issue tags: -Needs usability review

Sounds good.

Sam152’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs manual testing

Manually tested, looks great. Can't think of any reason this shouldn't be RTBC. Not sure anything would reasonably depend on the size of the icon.

webchick’s picture

Category: Feature request » Task
Status: Reviewed & tested by the community » Fixed

Nice!

Committed and pushed to 8.x. Thanks!

  • webchick committed 2ace268 on 8.0.x
    Issue #1883058 by cilefen, aendrew: Added Default Drupal drop favicon...

Status: Fixed » Needs work

The last submitted patch, 4: 1883058-4.patch, failed testing.

webchick’s picture

Status: Needs work » Fixed

Silly testbot.

cilefen’s picture

Issue tags: +needs backport to D7

Can anyone think of a reason not to backport this?

markcarver’s picture

Version: 8.0.x-dev » 7.x-dev
Status: Fixed » Active
Issue tags: -needs backport to D7

I do not see why not, go for it.

dcam’s picture

Status: Active » Patch (to be ported)

Setting the status so it will get noticed by patch-porters.

cilefen’s picture

Status: Patch (to be ported) » Needs review
FileSize
6.63 KB
PASSED: [[SimpleTest]]: [MySQL] 41,232 pass(es). View

D7 patch.

jecunningham2281’s picture

Status: Needs review » Reviewed & tested by the community

Tested in Drupal 7.34 Everything Looks good to me.

ChuChuNaKu’s picture

#SprintWeekend2015

mikemiles86’s picture

Issue tags: +SprintWeekend2015

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 21: 1883058-21.patch, failed testing.

Status: Needs work » Needs review

dcam queued 21: 1883058-21.patch for re-testing.

dcam’s picture

Status: Needs review » Reviewed & tested by the community

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 21: 1883058-21.patch, failed testing.

Status: Needs work » Needs review

David_Rothstein queued 21: 1883058-21.patch for re-testing.

David_Rothstein’s picture

Status: Needs review » Reviewed & tested by the community

Testbot fluke.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 21: 1883058-21.patch, failed testing.

Status: Needs work » Needs review

dcam queued 21: 1883058-21.patch for re-testing.

dcam’s picture

Status: Needs review » Reviewed & tested by the community
David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x - thanks!

  • David_Rothstein committed 9164413 on 7.x
    Issue #1883058 by cilefen, aendrew: Default Drupal "drop" favicon needs...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.