I'm using Views 3 with exposed sorts to display a carousel of nodes.

Scenario 1: I have set "Use pager" to "Display all items" in the view's settings so that all carousel items are preloaded on the page load. In this scenario, changing the sort order in the view's exposed form makes the carousel sort all of the items accordingly. This is good.

Scenario 2: I set a definite number of items to be preloaded in the carousel ("Use pager: 5 items"). This makes the carousel preload only 5 items and load any subsequent ones via Ajax. If I then choose a non-default sort order via the view's exposed form, the first 5 items in the carousel (that are preloaded) come out sorted according to the new sort order. However, any subsequent items that are loaded by the carousel via Ajax are still sorted according to the default one.

This needs to be corrected so that any Ajax-loaded items are ordered according to the currently selected order as well.

CommentFileSizeAuthor
#4 jcarousel.views_.inc_d7.patch1.38 KBbtibbils

Comments

quicksketch’s picture

This is pretty much the same issue as #601762: Exposed filter, where Exposed Filters have the same problem. Not sure what to recommend as a fix currently, but I'd be happy to look at patches (which should probably address both issues at the same time).

quicksketch’s picture

Title: Views 3 exposed sorts + Ajax-loaded carousel items order » jCarousel does not work with Exposed Filters or Sorts

Updating title for both issues.

quicksketch’s picture

The changes in #1095504: Make AJAX loading optional when "Use AJAX" is enabled in the view make it so that you can have a carousel that loads items via AJAX but the exposed filters and sorts do not use the ajax ability. This may help minimize the conflict, though this problem still isn't solved.

btibbils’s picture

StatusFileSize
new1.38 KB

The changes in the patch attached (to sites/all/modules/jcarousel/includes/jcarousel.views.inc) fixed the ignoring of exposed filters only and for D7.

Enjoy.

shevgeny’s picture

Issue summary: View changes

Error on line 130.

Notice: Undefined property: view::$exposed_input in function jcarousel_views_add() (line 130 in .../jcarousel.views.inc).

shevgeny’s picture

Status: Active » Needs work

Conflicting arguments