Eliminating wrap-arounds

[ckosloff]

I use 7zip (and have a six-year-old trial copy of Winzip:) so can't comment on how to use Winzip Pro 11... sorry...

I'll find out eventually, don't worry.
It's only that I am lazy and prefer to ask.
Thanks.

[ckosloff]

And finally here is THE additional images relocated with style contribution.
Feel free to tear it down, I will consider all suggestions.
Thanks.

[gjh42]

Without having actually tested it yet, I have just a few suggestions:

tpl_product_info_display.php looks fine, but it would be good to include a comment in the top section about the purpose and date of the specific change.

Everyone already has a modified stylesheet, and the stylesheet.css would erase their changes if carelessly uploaded.
It would be better to put only the modified stylesheet sections in a file named stylesheet_additional_images.css or something, so they could safely and easily be added/merged with the existing.

It would be helpful to put the date, title, version and other "header" info at the top of the readme. I would also suggest putting the whole CSS block there for easy reference, as it is not very large.
For consistency with other tags, it might be better to name the wrapper #additionalImagesWrapper, or perhaps #productImagesWrapper since it is for all the product images, not just additional ones.

/*beginning of additional images code*/

#additionalImageWrapper {
float: left;
width: 120px
}

#productAdditionalImages .additionalImages a {
float: left;
margin-left: 10px;
}

/*end of additional images code*/

One typo - "root/includes/templates/your_template/templates/tpl_product_display_info.php contains "

Good work so far, and I will test it when I have time tonight - I don't expect any difficulties.

[ckosloff]

Allrighty then.
Behold the birth of version 1.0 (open to further suggestions).
Thanks.

[gjh42]

Sorry for the long delay in replying. The package looks fine now, with one exception: you really should not have "stylesheet.css" in the includes - you should instead put the added declarations by themselves in a file called something like stylesheet_additional_images.css. You can include a comment that the code would best be added to the main stylesheet instead of loading another css file; but this will function just fine if uploaded alongside the user's stylesheet with no risk of adverse consequences.

I can guarantee that if you put stylesheet.css in the package, someone would ignore all instructions and upload it as is, erasing their own custom stylesheet (which they would not have backed up), and then come crying for help which we would not be able to give.

So go ahead and submit this mod, and thank you for doing this.

[blag]

Hear, hear!!

[ckosloff]

gotcha y'all.
I have been travelling for business the whole week so unable to answer in a timely manner, sorry about that.
Please note that the added declarations are already in the readme, with thorough explanations.
But caution is warranted, so instead of the standard stylesheet + the mod, I will create an additional stylesheet with just the mod, to be uploaded in same folder.
I will also add a comment at the top suggesting that it would be best to just add the code to the custom stylesheet instead of loading one too many, but that it would work either way.
Thank you so much for the input, I think this mod will clear many recurrent questions and stimulate creativity.

[ckosloff]

Oh, don't worry about the testing.
I have already loaded literally hundreds of pages, and they all work great both in IE and FF.
Later.

[Newmanz]

Hello,
I uploaded your files into the respective folders required.
When I go to load up a new product or edit and old product, I seem to be only getting the choice to add one picture file, same as before.

What have I done wrong?

I renamed my old display.php file original_info_display and uploaded tpl_product_info_display.php into templates/freeblue/templates

I uploaded the Additional_images_stylesheet.css into my templates/freeblue/css folder leaving the original stylesheet_new.css

www.newmanz.co.nz/zen

[ckosloff]

I just learned that v. 1.0 of this contribution is available for download in the templates section, it includes the latest modifications that we discussed here.
It has been reviewed by the Zen Cart team, and approved.
Hoorray!!!
Now, Newmanz, is this what you downloaded?
Second, your problem apparently has nothing to do with the files, Zen Cart will only allow you to upload ONE main product image.
The other images you upload manually, following the Zen Cart naming convention, into the same folder as main image.
e.g. MainImage_01, MainImage_02, etc.
You can also use alphabetical characters or a combination of both:
MainImage_a , MainImage_b, etc.
Always use just one underscore _, OK?
There is a lot more in the Manual pp. 203-207.
There is also a FAQ by Ajeh
https://www.zen-cart.com/tutorials/index.php?article=58
After you understand that, you can use Image Handler to upload additional images for you.
Dig into that first, and then be back to us if problems.
Hope this helps.