Make WordPress Core

Opened 10 years ago

Closed 10 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 10 years ago.
twentyfourteen-gallery-display-table-bug.png (153.2 KB) - added by celloexpressions 10 years ago.
Before/After patch on IE11
25625.2.diff (572 bytes) - added by iamtakashi 10 years ago.

Download all attachments as: .zip

Change History (10)

@jartes
10 years ago

#1 @jartes
10 years ago

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

#2 @celloexpressions
10 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
10 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
10 years ago

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

@celloexpressions
10 years ago

Before/After patch on IE11

#5 @valeriosza
10 years ago

  • Cc valeriosza@… added

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

@iamtakashi
10 years ago

#6 @iamtakashi
10 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
10 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.