Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#34433 closed defect (bug) (fixed)

Embed template: Site icon should have an alternative for HiDPI screens

Reported by: ocean90's profile ocean90 Owned by: ocean90's profile ocean90
Milestone: 4.4 Priority: normal
Severity: normal Version: 4.4
Component: Embeds Keywords: has-patch commit
Focuses: Cc:

Description

The template uses the 32px version of the site icon. The next available size is 150px. We should either load the 150px for all screens or use the srsetattribute, see attached patch.

Attachments (3)

34433-before.png (18.4 KB) - added by ocean90 8 years ago.
34433-after.png (19.7 KB) - added by ocean90 8 years ago.
34433.patch (1.5 KB) - added by ocean90 8 years ago.

Download all attachments as: .zip

Change History (5)

@ocean90
8 years ago

@ocean90
8 years ago

@ocean90
8 years ago

#1 @pento
8 years ago

  • Keywords commit added
  • Owner set to ocean90
  • Status changed from new to assigned

Agreed.

#2 @ocean90
8 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 35409:

Embeds: Provide a site icon via srcset for HiDPI screens.

Fixes #34433.

Note: See TracTickets for help on using tickets.