Solved

Only displaying 3 images per row

Posted on 2004-09-26
8
160 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
8 Comments
 
LVL 25

Accepted Solution

by:
James Rodgers earned 500 total points
Comment Utility
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
Comment Utility
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
Comment Utility
that could be due to your cell spacing or cell padding properties of Table.
0
 
LVL 10

Expert Comment

by:Mause
Comment Utility
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
How your wiki can always stay up-to-date

Quip doubles as a “living” wiki and a project management tool that evolves with your organization. As you finish projects in Quip, the work remains, easily accessible to all team members, new and old.
- Increase transparency
- Onboard new hires faster
- Access from mobile/offline

 

Expert Comment

by:tpiazza
Comment Utility
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
Comment Utility
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
Comment Utility
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
Comment Utility
glad i could help

thanks for the points
0

Featured Post

Free Gift Card with Acronis Backup Purchase!

Backup any data in any location: local and remote systems, physical and virtual servers, private and public clouds, Macs and PCs, tablets and mobile devices, & more! For limited time only, buy any Acronis backup products and get a FREE Amazon/Best Buy gift card worth up to $200!

Join & Write a Comment

Have you ever sent email via ColdFusion and thought of tracking this mail to capture the exact date and time when the message was opened ?  If yes, then this article is for you ! First we need a table user_email with columns user_id , email , sub…
Meet the world's only “Transparent Cloud™” from Superb Internet Corporation. Now, you can experience firsthand a cloud platform that consistently outperforms Amazon Web Services (AWS), IBM’s Softlayer, and Microsoft’s Azure when it comes to CPU and …
Polish reports in Access so they look terrific. Take yourself to another level. Equations, Back Color, Alternate Back Color. Write easy VBA Code. Tighten space to use less pages. Launch report from a menu, considering criteria only when it is filled…
This tutorial demonstrates a quick way of adding group price to multiple Magento products.

772 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

Need Help in Real-Time?

Connect with top rated Experts

11 Experts available now in Live!

Get 1:1 Help Now