Display more than one item at once (grid/table)

upupax - June 8, 2009 - 10:11
Project:Views Slideshow
Version:6.x-2.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs work
Description

Hi, I'm using VSS to rotate news in my site. Having a tall block, i would like to display 2 nodes at once: is it possible?

#1

naught101 - July 2, 2009 - 08:11

I'm also interested in this. I want to try to re-create this kind of effect: http://www.abpat.qld.edu.au/

At the moment, I can stack four images with a grid view, or I can display one with a slideshow. I figure I could also use minipanels to create a four column block to show four offset slideshows, but then I run the risk of having duplicates, since the views would all be separate, and wouldn't be able to check for uniqueness. Not to mention panels+views memory hit.

A "slideshow grid" would probably be ideal for me, and I guess that could work for you too upupax: a grid with 1 column and two rows.

#2

a.luiz.n - July 7, 2009 - 19:48

i really need this...any help?

#3

a.luiz.n - July 7, 2009 - 19:59

i'm using an attachment view with exactly the same setup as the block, applying it to the block and using sort criteria as the opossite....

is there a better way?

#4

naught101 - July 10, 2009 - 05:23
Title:Display more than one item at once» Display more than one item at once (grid/table)

I think this might be two (linked) features in one:
First, a normal slideshow, in which each slide is composed of multiple slides (like a mini-panel, or a mini-view?). On each change, all nodes displayed would change to another set

Second: a gridded slideshow, in which each panel is it's own slideshow, so that each panel can change at different times, and cycle between different sets of nodes. - ie something like a view made up of multiple slideshows. (this would allow something like the link which I posted above)

They would both be visually the same if the timer was set to go at the same time for each, which would probably be the default value. The second option may be overkill anyway...

Which of the two options do others want most?

#5

upupax - July 11, 2009 - 00:33

I'm most interested in the first one..

#6

a.luiz.n - July 13, 2009 - 17:53

i like the first one cause there wouldn't be repeated nodes.

#7

naught101 - July 28, 2009 - 11:45
Priority:normal» minor

As a work around: It's relatively easy to create a few nodes that have two parts, and then just use those multi-part nodes in a views slideshow.

This Feature request wouldn't add much to that approach. It would make it slightly easier to set up, but would give less control than the work around (because with the work around you can use specific css/tables/whatever to make the layout look exactly as you want).

On the other hand, it could be cool once Views_Slideshow gets a few more transitions to combine them in a grid - ie. you have two panes, the first one fades out, the other one slides across, and a third one fades in where the second one was previously.

#8

redndahead - August 26, 2009 - 05:17
Version:6.x-1.x-dev» 6.x-2.x-dev

Moving to 2.x

#9

hades666evil - October 2, 2009 - 00:04
Version:6.x-2.x-dev» 6.x-2.0-beta2
Priority:minor» normal
Status:active» needs review

I don't know it it's exactly what you wanted to do, but I've made some changed to the Views Slideshow: SingleFrame module so that it can display multiple items per slide.
Basically, it just adds an item_per_slide parametter (in views_slideshow_singleframe.views_slideshow.inc) and few changes in views_slideshow_singleframe.theme.inc

theres some changes in theses functions :
- theme_views_slideshow_singleframe_no_display_section
- theme_views_slideshow_singleframe_no_display_teaser
- views_slideshow_singleframe_views_slideshow_options_form

AttachmentSize
views_slideshow_singleframe_multiple_items_patch.zip 4.09 KB

#10

redndahead - October 2, 2009 - 06:43
Version:6.x-2.0-beta2» 6.x-2.x-dev
Status:needs review» needs work

Could you please make a regular patch and attach it. You can find instructions here: http://drupal.org/patch/create

#11

hades666evil - October 2, 2009 - 12:29

Alors j'ai essayé de créer mon premier patch avec winmerge, voilà ce que ça donne (j'ai repris à partir de la version dev, du coup, je crois que c'est mieux pour les patchs)

AttachmentSize
views_slideshow_singleframe_patch.zip 1.77 KB

#12

redndahead - October 2, 2009 - 15:36

Next time please in english, I don't know french but I got the idea using a translation service.

That's close. There are some issues with it like line endings and such. Make sure it matches drupal coding standards also. http://drupal.org/coding-standards

I would follow this tutorial on how to install cygwin correctly as it produces better patches. http://www.lullabot.com/videocast/install-cygwin-windows-xp

Please don't get too frustrated with the steps I have you go through to submit a patch. It just helps you become a better drupal coder.

#13

hades666evil - October 3, 2009 - 14:18

Sorry for the french, I've just forgotten to switch
For my next patch, I'll make it with the debian installed on my local serveur.
Thanks for the advises, I'll try to follow them.

#14

intyms - November 9, 2009 - 18:18

thank you, subscribing...

#15

hades666evil - November 10, 2009 - 19:34

I've updated my my patch to resolve some bugs (the slideshow did not appears if there was less than one full slide)
This time I've tried to use the diff command on debian and used a better syntax

AttachmentSize
views_slideshow_singleframe_multipleitems.patch 4.45 KB

#16

redndahead - November 10, 2009 - 21:48

+++ views_slideshow/contrib/views_slideshow_singleframe/views_slideshow_singleframe.theme.inc 2009-11-10 20:11:27.000000000 +0100
@@ -51,10 +51,27 @@ function theme_views_slideshow_singlefra
+  //retrive the number of itmes per frame

should be: retrieve the number of items per frame

+++ views_slideshow/contrib/views_slideshow_singleframe/views_slideshow_singleframe.theme.inc 2009-11-10 20:11:27.000000000 +0100
@@ -51,10 +51,27 @@ function theme_views_slideshow_singlefra
+  for ($i=0;$i<count($rows);$i++) {

spacing: for ($i = 0; $i < count($rows); $i++) {

+++ views_slideshow/contrib/views_slideshow_singleframe/views_slideshow_singleframe.theme.inc 2009-11-10 20:11:27.000000000 +0100
@@ -51,10 +51,27 @@ function theme_views_slideshow_singlefra
+    if(count($items) == $items_per_slide || $i == (count($rows)-1) ) {
@@ -64,20 +81,28 @@ function theme_views_slideshow_singlefra
+  if ($slidecount>1) {

spacing: if ($slidecount > 1) {

+++ views_slideshow/contrib/views_slideshow_singleframe/views_slideshow_singleframe.theme.inc 2009-11-10 20:11:27.000000000 +0100
@@ -64,20 +81,28 @@ function theme_views_slideshow_singlefra
+  foreach($items as $itemcount => $item){

spacing: foreach ($items as $itemcount >= $item) {

+++ views_slideshow/contrib/views_slideshow_singleframe/views_slideshow_singleframe.theme.inc 2009-11-10 20:11:27.000000000 +0100
@@ -64,20 +81,28 @@ function theme_views_slideshow_singlefra
+    $itemclass = 'views-row views-row-'.$itemcount;

spacing: $itemclass = 'views-row views-row-' . $itemcount;

+++ views_slideshow/contrib/views_slideshow_singleframe/views_slideshow_singleframe.theme.inc 2009-11-10 20:11:27.000000000 +0100
@@ -64,20 +81,28 @@ function theme_views_slideshow_singlefra
+    if(!$itemcount){

spacing: if (!$itemcount) {

+++ views_slideshow/contrib/views_slideshow_singleframe/views_slideshow_singleframe.theme.inc 2009-11-10 20:11:27.000000000 +0100
@@ -64,20 +81,28 @@ function theme_views_slideshow_singlefra
+    $output .= '    <div class="'.$itemclass.'">'. "\n";

spacing: $output .= '' . "\n";

Ran out of time for review and am not reviewing the functionality yet, but two word variables should be separated by an underscore and check your spacing in function names and in between "." in concantenation. Also for, if etc. should have a space between the "for" and the "(". Thanks for submitting a patch.

This review is powered by Dreditor.

#17

hades666evil - November 12, 2009 - 15:50

Hi,
This is a news version of the patch with the lastest dev (2009-Nov-06).
I've tried to follow your advises for the text formating.

AttachmentSize
views_slideshow_singleframe_multipleitems.patch 4.57 KB
 
 

Drupal is a registered trademark of Dries Buytaert.