The scroll to top button does not appear in IE8 while scrolling down.
It seems that $(this).scrollTop() > 100 is always false.
Perhaps related to ScrollTop not working in IE.

Attaching a patch, which tests if document.documentElement.scrollTop exists, and use it as a fallback.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jfhovinne’s picture

Here is the patch.

jfhovinne’s picture

Status: Active » Needs review
tarekdj’s picture

Good job jfhovinne. The code looks good for me. I wonder if you can provide some before/after screenshots.
Thank you again.

jfhovinne’s picture

Sorry, screenshots not available.
Before: no button as it is hidden by $("#back-top").hide();.
After: button is displayed while scrolling down.

tarekdj’s picture

Status: Needs review » Reviewed & tested by the community
tarekdj’s picture

Status: Reviewed & tested by the community » Fixed

commited to 7.x-2.x! Merci jfhovinne :)

tarekdj’s picture

Version: 7.x-2.x-dev » 6.x-2.x-dev
Status: Fixed » Patch (to be ported)
Issue tags: +Novice, +TUNIS_2014_FEBRUARY

Issue marked for our Tunisian local sprint next week :)

willieseabrook’s picture

Issue tags: +TUNIS_2014_MARCH

  • tarekdj committed 896f854 on 8.x-0.x
    Issue #2199267 by jfhovinne: Button does not appear in IE8.