Closed (fixed)
Project:
Views Infinite Scroll
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Oct 2011 at 19:34 UTC
Updated:
24 Mar 2016 at 18:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
aaronbaumanAccording to some other comments, this was supposed to be written already, but I couldn't find it anywhere.
On top of that the "manual example" on the example website was not, in fact, manual.
Here's a patch that applies to the latest DEV.
Comment #2
quadbyte commentedThanks Aaron for this excellent feature addition, just tested your patch and it's working fine.
However I found a way to improve it a little by simply changing the button label option key from "manual_load_label" to "text" which seems to be more compliant with Views and allowed me to get support for arguments title; i.e I needed the label to read "Show more sports", "Show more politic" automatically.
Comment #3
quadbyte commentedFixed a typo in views_plugin_pager_infinite_scroll.inc
$options['maunal_load'] = array('default' => 0);
Should be
$options['manual_load'] = array('default' => 0);
Also my previous comment was not really accurate, the change from 'manual_load_label' to 'text' is not necessary I was mislead by some side effect with some existing customization we had on our site, sorry about that.
Comment #4
ChrisConolly commentedany chance of this feature making an appearance on Drupal 7?
Comment #5
betovargHi guys, im a total newb at patching. Is there anyway someone can point me to a downloadable module that has this patch applied?
Does anyone know if this will make it into the module?
Thanx
Comment #6
bryancasler commentedGreat patch, sorely needed functionality.
Comment #7
codigovision commentedThanks for the patch, this is exactly what I was looking for! I made a small addition to wrap the button in a div with id so it can be easily styled in the theme.
Just added this to views_infinite_scroll.js:
Comment #8
codigovision commentedHere's the patch, adapted from #3
Comment #9
sarath.rajan commentedIs there any patch for D7? :(
Comment #10
GloryFish commentedI rolled a patch based on comment #8 for Drupal 7, it's here: #1942142: Patch to manually load subsequent pages for Drupal 7
Comment #11
jmuzz commentedWhoops wrong thread.
Comment #12
jmuzz commentedMy bad.
Comment #14
sam152 commentedThank you for the patch, this fix has been fixed in the last dev release and will likely be tagged in a stable release over the next few weeks.
Comment #16
joseph.olstadFixed in 7.x-1.2-dev , if you need this functionality, upgrade to 7.x-1.2-dev