Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#25625 closed defect (bug) (fixed)

Twenty Fourteen: Gallery Images Initially Load Small in IE

Reported by: celloexpressions's profile celloexpressions Owned by: lancewillett's profile lancewillett
Milestone: 3.8 Priority: low
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

In IE 8-10, galleries are loading with images displaying at about 50% of their intended size. As soon as the gallery is hovered, the display fixes itself (that part varies by version). I have no idea why.

Attachments (3)

25625.diff (346 bytes) - added by jartes 13 years ago.
twentyfourteen-gallery-display-table-bug.png (153.2 KB) - added by celloexpressions 13 years ago.
Before/After patch on IE11
25625.2.diff (572 bytes) - added by iamtakashi 13 years ago.

Download all attachments as: .zip

Change History (10)

@jartes
13 years ago

#1 @jartes
13 years ago

  • Cc joan@… added
  • Keywords has-patch added

#2 @celloexpressions
13 years ago

Well, that fixes it (and doesn't break in Chrome or Firefox). Noting now that the bug didn't in fact show up on galleries with more than three columns. It is also present in IE11, but different (stays small regardless of hover).

One note: patches should always be relative to the WordPress root; so the path to style.css would be wp-content/themes/twentyfourteen/ in this case.

#3 @lancewillett
13 years ago

For visual bugs can you please attach before / after screenshots or screencasts? It's a big help for reports like this.

#4 @lancewillett
13 years ago

  • Milestone changed from Awaiting Review to 3.8
  • Priority changed from normal to low

@celloexpressions
13 years ago

Before/After patch on IE11

#5 @valeriosza
13 years ago

  • Cc valeriosza@… added

Strange, for some strange reason it works perfectly with me.

@iamtakashi
13 years ago

#6 @iamtakashi
13 years ago

I can confirm the bug on IE. The purpose of that line was centering a gallery when its width is most likely narrower than the content width. with/without comparison. But that's causing a greater issue on IE and it doesn't look so bad even it's not centered so we can remove it. Attached a refreshed patch above.

#7 @lancewillett
13 years ago

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

In 26057:

Twenty Fourteen: fix gallery layout issue in IE, props jartes and iamtakashi. Fixes #25625.

Note: See TracTickets for help on using tickets.