Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hugronaphor’s picture

I applied your patch, and it's working great.
Thanks.

maximilianmikus’s picture

works fine! Thank you!

kolier’s picture

Status: Active » Needs review

Works well +1.

dafeder’s picture

Working for me as well so far with no issues.

jmuzz’s picture

Very nice, but the text shouldn't be required. I want to give the button an image using a background instead of using text in the link. It would be strange for something in a views form to be required, most of the elements are not, including the text for the buttons in the other pagers.

corbacho’s picture

+++ b/js/views_infinite_scroll.js
@@ -35,31 +37,50 @@ Drupal.behaviors.views_infinite_scroll = {
+                return false;

return false; in the click handler, is preventing to have any other event listener for the click. I replaced it with e.preventDefault();
The change is on top of the patch of jmuzz (#5)

Sam152’s picture

Status: Needs review » Fixed

This has been added to the dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.