Solved

Only displaying 3 images per row

Posted on 2004-09-26
8
176 Views
Last Modified: 2013-12-24
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
0
Comment
Question by:bluskyGuy
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
8 Comments
 
LVL 25

Accepted Solution

by:
James Rodgers earned 500 total points
ID: 12156120
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
 

Author Comment

by:bluskyGuy
ID: 12156138
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
 
LVL 21

Expert Comment

by:pinaldave
ID: 12156264
that could be due to your cell spacing or cell padding properties of Table.
0
Optimize your web performance

What's in the eBook?
- Full list of reasons for poor performance
- Ultimate measures to speed things up
- Primary web monitoring types
- KPIs you should be monitoring in order to increase your ROI

 
LVL 10

Expert Comment

by:Mause
ID: 12157896
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
 

Expert Comment

by:tpiazza
ID: 12161100
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
 
LVL 9

Expert Comment

by:digicidal
ID: 12186678
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
 
LVL 2

Expert Comment

by:KoldFuzun
ID: 12202213
try also adding a <br /> tag after your image tags... right before the </td>

This will tighten up your table
0
 
LVL 25

Expert Comment

by:James Rodgers
ID: 12251270
glad i could help

thanks for the points
0

Featured Post

Turn your laptop into a mobile console!

The CV211 Laptop USB Console Adapter provides a direct Laptop-to-Computer connection for fast and easy remote desktop access with no software to install.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

In our day to day coding, how many times have we come across a necessity to check whether a URL is a broken link or not? For those of you that answered countless and are using ColdFusion like myself, then this article is for you.  It will show yo…
One of the typical problems I have experienced is when you have to move a web server from one hosting site to another. You normally prepare all on the new host, transfer the site, change DNS and cross your fingers hoping all will be ok on new server…
In this video, viewers will be given step by step instructions on adjusting mouse, pointer and cursor visibility in Microsoft Windows 10. The video seeks to educate those who are struggling with the new Windows 10 Graphical User Interface. Change Cu…
In this video we outline the Physical Segments view of NetCrunch network monitor. By following this brief how-to video, you will be able to learn how NetCrunch visualizes your network, how granular is the information collected, as well as where to f…

628 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question