Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#22681 closed defect (bug) (fixed)

Don't blow up small thumbnails

Reported by: nacin's profile nacin Owned by: koopersmith's profile koopersmith
Milestone: 3.5 Priority: low
Severity: minor Version:
Component: Media Keywords: has-patch commit
Focuses: Cc:

Description

Here's a favicon forced to be many more pixels wide than it should be: http://cl.ly/image/460U083s343M. Same thing happens with small images (not just .ico files). I imagine this is a fairly straightforward CSS change.

Attachments (1)

22681.diff (515 bytes) - added by helenyhou 13 years ago.

Download all attachments as: .zip

Change History (5)

#1 @pavelevap
13 years ago

Yes, I noticed the same problem before: http://core.trac.wordpress.org/ticket/22518#comment:2

But I forget to promote it...

@helenyhou
13 years ago

#2 @helenyhou
13 years ago

  • Keywords has-patch added; needs-patch needs-body removed

22681.diff seems to work - tested Firefox and Chrome on OSX and IE7 and IE8 in XP on Browserstack. IE doesn't center the images - probably because of the transform used. I don't think it's a wrecky experience for those browsers, but can look at doing more with it if so desired.

#3 @nacin
13 years ago

  • Keywords commit added

Solid to me.

#4 @koopersmith
13 years ago

  • Owner set to koopersmith
  • Resolution set to fixed
  • Status changed from new to closed

In 22978:

Prevent small thumbnails from being upscaled. props helenyhou. fixes #22681.

Note: See TracTickets for help on using tickets.