Only displaying 3 images per row

I have a query that's displaying images. The problem is all the images are being displayed on one row causing the user to have to scroll to the right.

What I would like to do is have the images displayed 3 images per row. So, if there are 15 images, there would be 5 rows with 3 images in each row.

Here's the output that's currently outputting all images on one row.

<cfset variables.opt_count = 0>
                                          <cfloop query="getoption">
                                                <cfif getOption.image neq 0>
                                                      <td class="formText" align="center"><a href="##" onclick="document.order_form.product_option_#attributes.count#[#variables.opt_count#].selected=true;" class="formText"><img src="#request.urlRoot##request.relativePath#images/site_images/#getOption.image#" alt="#getOption.item#" border="0"><br>#getOption.currentrow#</a></td>
                                                </cfif>
                                                <cfset variables.opt_count = #variables.opt_count# + 1>
                                                
                                          </cfloop>

Thanks,
Rick
bluskyGuyAsked:
Who is Participating?
 
James RodgersConnect With a Mentor Web Applications DeveloperCommented:
try this

<cfset variables.opt_count = 0>
                                   <cfloop query="getoption">
                                        <cfif getOption.image neq 0>
                                             <td class="formText" align="center"><a href="##" onclick="document.order_form.product_option_#attributes.count#[#variables.opt_count#].selected=true;" class="formText"><img src="#request.urlRoot##request.relativePath#images/site_images/#getOption.image#" alt="#getOption.item#" border="0"><br>#getOption.currentrow#</a></td>
                                        </cfif>
                                        <cfset variables.opt_count = #variables.opt_count# + 1>
                                        <cfif not getOption.currentRow mod 3>
</tr>
<tr>
</cfif>
                                   </cfloop>
0
 
bluskyGuyAuthor Commented:
Thanks for the fast response. Yea, the images are now being displayed with 3 images per row. however, there is a large white space above the rows of images where there was none before? Any ideas?

Thanks,
Rick
0
 
pinaldaveCommented:
that could be due to your cell spacing or cell padding properties of Table.
0
Introducing Cloud Class® training courses

Tech changes fast. You can learn faster. That’s why we’re bringing professional training courses to Experts Exchange. With a subscription, you can access all the Cloud Class® courses to expand your education, prep for certifications, and get top-notch instructions.

 
MauseCommented:
Hi there

Try this, its almost the same as Jester.
Also make sure the <table> tags and the first <tr> tags are placed just before and after the loop like this:

<table>
<tr>
<cfloop query="getoption">
     <cfif getOption.image neq 0>
          <td class="formText" align="center"><a href="##" onclick="document.order_form.product_option_#attributes.count#[#variables.opt_count#].selected=true;" class="formText"><img src="#request.urlRoot##request.relativePath#images/site_images/#getOption.image#" alt="#getOption.item#" border="0"><br>#getOption.currentrow#</a></td>
     </cfif>
     <cfset variables.opt_count = #variables.opt_count# + 1>

     <cfif not getOption.currentRow mod 3 AND getOption.currentRow neq getOption.recordcount>
            </tr>
            <tr>
      </cfif>
</cfloop>
</tr>
</table>


Mause
0
 
tpiazzaCommented:
this is what i do

<table>

<cfset counter ="0">
<tr>
<cfoutput query="whatever">
<cfif counter is not "3">
<td>image code</td>
<cfset counter = counter +1>
<cfelse>
<td>image code</td>
</tr>
<tr>
<cfset counter = "0">
</cfif>
</cfoutput>

</table>
0
 
digicidalCommented:
Just a suggestion, but not really necessary for you, what if you used CSS instead of tables... by using DIV's and float:left or SPANs properties of CSS you could eliminate the table altogether.  Since you're using the class attribute for your TD's you obviously have a stylesheet defined already.  Here would be one way of achieving this:


<div id="picureDisplayArea">
<div>
<cfloop query="getoption">
     <cfif getOption.image neq 0>
          <span class="formText"><a href="##" onclick="document.order_form.product_option_#attributes.count#[#variables.opt_count#].selected=true;" class="formText"><img src="#request.urlRoot##request.relativePath#images/site_images/#getOption.image#" alt="#getOption.item#" border="0"><br>#getOption.currentrow#</a></span>
     </cfif>
     <cfset variables.opt_count = #variables.opt_count# + 1>

     <cfif not getOption.currentRow mod 3 AND getOption.currentRow neq getOption.recordcount>
          </div>
          <div>
     </cfif>
</cfloop>
</div>
</div>

Although this will basically work exactly the same as the table example it has a couple of added benefits... for example if you are loading a large number of records, a DIV will display as soon as the content is finished... a TABLE is the same way, but this means that your table will not be displayed until the entire table is loaded.  This way as soon as a picture is loaded, it is displayed.  Didn't actually try this exact example, just translated on the above examples by Mause and Jester (don't give me points unless you give them 90% of the points on this question - I'm just suggesting an alternative method).

Food for thought... if you really need me to, or if you have problems, add another comment and I'll go ahead and do the CSS definitions and everything with test data to make sure it runs properly.

Cheerz.
0
 
KoldFuzunCommented:
try also adding a <br /> tag after your image tags... right before the </td>

This will tighten up your table
0
 
James RodgersWeb Applications DeveloperCommented:
glad i could help

thanks for the points
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.