Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Infinite Scroll: infinite-footer HTML at bottom of page when Infinite Scroll is not activated #5268

Closed
grdkly opened this issue Oct 5, 2016 · 0 comments
Labels
[Feature] Infinite Scroll [Pri] Low [Type] Bug When a feature is broken and / or not performing as intended

Comments

@grdkly
Copy link

grdkly commented Oct 5, 2016

Steps to reproduce the issue

  1. Create blog with only one post, e.g. fresh install of WP 4.6.1
  2. Install Jetpack 4.3.1, enable Infinite Scroll with default settings.
  3. Scroll to bottom of front page.

What I expected

Infinite Scroll is not activated.

What happened instead

Infinite Scroll HTML <div id="infinite-footer"> is included at bottom of page even though Infinite Scroll script and style have not been included.

Link to example

http://grdkly.com/wordpress/

Possible root cause

infinity.php: Testing is_last_batch() in function action_template_redirect() correctly prevents inclusion of Infinite Scroll script and style; a similar test is needed in function footer() to prevent inclusion of Infinite Scroll footer HTML <div id="infinite-footer">. The test was changed in #3553.

@jeherve jeherve added [Type] Bug When a feature is broken and / or not performing as intended [Feature] Infinite Scroll [Team] tdiv labels Oct 6, 2016
@jeherve jeherve added this to the 4.4 milestone Oct 6, 2016
@jeherve jeherve modified the milestones: Not Currently Planned, 4.4 Nov 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Infinite Scroll [Pri] Low [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

No branches or pull requests

2 participants