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.

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.
    
astonvictor’s picture

Status: Patch (to be ported) » Closed (outdated)

D6 reached its EOL back in February 2016, and there is no active release for D6 for this module anymore.
Development or support is not planned for D6. All D6-related issues are marked as outdated in a bunch.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.