Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#31734 closed defect (bug) (fixed)

Twenty Fifteen flattering with Twitterfeed

Reported by: derelch's profile derElch Owned by: lancewillett's profile lancewillett
Milestone: 4.2 Priority: normal
Severity: normal Version: 4.1.1
Component: Bundled Theme Keywords: 2nd-opinion has-patch
Focuses: Cc:

Description

This is a follow-up to #29987.

i created here a GIF Animation which shows the trouble.
if i'm using the official Twitter Feed (included by Twitter JS Code or Jetpack Plugin doesn't make a difference) it begins to falter while scrolling down.

pls see: http://temp.elchalarm.net/twitterfalter.gif

Problem can be comprehended at rideSonic.ch
(WordPress 4.1, Jetpack 3.3, TwentyFifteen 1.0)

Theme is only modified in small parts but the bug exists in the main theme too. Bug discovered in Firefox 34.0.05 and IE 11.0.96, in Google Chrome it works as it should (mostly). Is this a real bug or just something which the browser (FF/IE) can't handle?

an "other" bug:
if scrolling down, it stops above the page bottom. after scrolling forward/downward it beginns to flatter. this happens mostly after visiting a page with less content (not so large pages) and/or if i beginn immediately after page loading to scroll.

pls see: http://temp.elchalarm.net/wp-2015-flatter.gif

Thank you very much and have a nice day.

Attachments (1)

31734.diff (673 bytes) - added by jartes 10 years ago.

Download all attachments as: .zip

Change History (6)

#1 @lancewillett
10 years ago

  • Keywords 2nd-opinion added
  • Milestone changed from Awaiting Review to 4.2

Can anyone else repeat, please? Please note your OS version and browser version -- using "default" WP install with unmodified Twenty Fifteen theme, if possible.

Last edited 10 years ago by lancewillett (previous) (diff)

This ticket was mentioned in Slack in #core-themes by lancewillett. View the logs.


10 years ago

@jartes
10 years ago

#3 @jartes
10 years ago

  • Keywords has-patch added

In 31734.diff moved the calculation of the sidebar height to the scroll() function to calculate the sidebar height every time we scroll. Now, this is calculated in the beginning, when the dynamic widgets with iframes, like Twitter, are not rendered yet.

#4 @jartes
10 years ago

As the reporter said the error was visible only in FF and IE. I was able to reproduce the error in FF 36.0.4 (Mac) with my local install at the 31912 trunk revision: https://cloudup.com/cu2ghhv1dSv.

#5 @lancewillett
10 years ago

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

In 31934:

Twenty Fifteen: adjust scroll behavior to make sure sidebar height is calculated during scroll events, not just resize events.

Fixes #31734. Props jartes, lancewillett.

Note: See TracTickets for help on using tickets.