Zen Colorbox

[davewest]

You know that jQuery is getting called for already in ZC 1.5.4!

line 108 of /inculdes/templates/template_default/common/html_header.php

/** CDN for jQuery core **/ 
?> 
  <script>window.jQuery || document.write('<script src="//code.jquery.com/jquery-1.11.1.min.js"><\/script>');</script> 
  <script>window.jQuery || document.write('<script src="<?php echo $template->get_template_dir('.js',DIR_WS_TEMPLATE, $current_page_base,'jscript'); ?>/jquery.min.js"><\/script>');</script> 
 
<?php

of coruse if a custom template is not updated to 1.5.4, then it may be missing...

I've been doing popups using the standard popup and Jacks' iFrame method... finished setting up the contact us form using one of the responsive templates, but got side track so haven't finished setting up the others yet.

[mc12345678]

You know that jQuery is getting called for already in ZC 1.5.4!

line 108 of /inculdes/templates/template_default/common/html_header.php

/** CDN for jQuery core **/ 
?> 
  <script>window.jQuery || document.write('<script src="//code.jquery.com/jquery-1.11.1.min.js"><\/script>');</script> 
  <script>window.jQuery || document.write('<script src="<?php echo $template->get_template_dir('.js',DIR_WS_TEMPLATE, $current_page_base,'jscript'); ?>/jquery.min.js"><\/script>');</script> 
 
<?php

of coruse if a custom template is not updated to 1.5.4, then it may be missing...

I've been doing popups using the standard popup and Jacks' iFrame method... finished setting up the contact us form using one of the responsive templates, but got side track so haven't finished setting up the others yet.

That's one of the reasons for the che check to see if it is already installed . Another is that this code still supports versions before ZC 1.5.4, so either force everyone that uses it to upgrade to ensure up-to-date security or take minor action to continue support of older versions and potentially incorrectly installed jquery script...

[DivaVocals]

Okie-dokie. I'll play with the additional pop-ups and report back. Meanwhile, I made the following changes to /includes/templates/YOUR_TEMPLATE/jscript/jscript_jquery_min.php to correct the HTML validation errors:

<?php
//Load jQuery ONLY if jQuery has not been loaded. This is to support versions of Zen Cart prior to v1.5.4 because jQuery was not part of core code
//Also want to prevent jQuery from loading again if bundled with a plugin or template package
?>
<script type="text/javascript">
if (typeof jQuery == 'undefined') {
    document.write(unescape('%3Cscript type="text/javascript" src="//code.jquery.com/jquery-1.11.1.min.js"%3E%3C/script%3E'));
}
</script>

Other than that, all looked good. Tested on a fresh ZC1.5.4 install, no IH4, PHP 5.4.35.

You know that jQuery is getting called for already in ZC 1.5.4!

line 108 of /inculdes/templates/template_default/common/html_header.php

/** CDN for jQuery core **/ 
?> 
  <script>window.jQuery || document.write('<script src="//code.jquery.com/jquery-1.11.1.min.js"><\/script>');</script> 
  <script>window.jQuery || document.write('<script src="<?php echo $template->get_template_dir('.js',DIR_WS_TEMPLATE, $current_page_base,'jscript'); ?>/jquery.min.js"><\/script>');</script> 
 
<?php

of coruse if a custom template is not updated to 1.5.4, then it may be missing...

I've been doing popups using the standard popup and Jacks' iFrame method... finished setting up the contact us form using one of the responsive templates, but got side track so haven't finished setting up the others yet.

Very aware of the jQuery inclusion.. as mc12345678 pointed out, and as indicated by the code comment it is to support older v1.5 versions..

Regarding the popups, if you'd like to fork my repo and contribute lightbox popups that would be awesome..

[davewest]

Very aware of the jQuery inclusion.. as mc12345678 pointed out, and as indicated by the code comment it is to support older v1.5 versions..

Regarding the popups, if you'd like to fork my repo and contribute lightbox popups that would be awesome..

Ya... I kind of fugued you already knew... one thing I like about the ZC154 one, if your testing and not live, it looks to the template folder for one to load...

I was looking at both GitHubs, but didn't see any popups, so was wondering what method you was using. If the same, maybe save you some work..

[DivaVocals]

Ya... I kind of fugued you already knew... one thing I like about the ZC154 one, if your testing and not live, it looks to the template folder for one to load...

Yep.. in fact the new version I'll being submitting even fixes the auto installer in versions prior to v.1.5.x so the admin menu part of the install doesn't error out..

I was looking at both GitHubs, but didn't see any popups, so was wondering what method you was using. If the same, maybe save you some work..

Not my method at all.. WAYYYYYY over my head.. The popups are in Daniels repo..

My repo (https://github.com/DivaVocals/zen-colorbox) is a fork of Daniel's (https://github.com/daniel-hopkins/zen-colorbox). My fork includes all the latest updates to support Zen Cart v1.5.4.. As Daniel recently merged my pull request with all of the Zen Cart v1.5.4 updates, now his fork contains the same updates.. Where our repo's DIFFER is the code Daniel included to support the popups. That code is in the includes/modules/pages folder (https://github.com/daniel-hopkins/ze.../modules/pages)

If you have a complete/better/working method, then PLEASE contribute.. I've updated the issue log with the problems found in testing the code Daniel started working on..

[duluthpopsshop]

Product image problem - - - ZC1.5.4 fresh - - Responsive Basic Blue - - - Zen_Colorbox -- browser, Chrome
Category listing problem page - http://www.gopopsshop.com/index.php?...ndex&cPath=7_8
Product Listing page is OK - http://www.gopopsshop.com/index.php?...&products_id=7

My problem is related to Colorbox because pages were OK before Colorbox.

I am unable to find where the size of the images are controlled.
On the Categorys page, the pix are sized to 98 x 80 px - skinny and tall - wrong orientation
On the Products page, the pix are sized to 147 x 120 px - looks good

What is overriding the code on the Category page that causes the picture size to change so dramatically????

Looking forward to suggestions

[DivaVocals]

Product image problem - - - ZC1.5.4 fresh - - Responsive Basic Blue - - - Zen_Colorbox -- browser, Chrome
Category listing problem page - http://www.gopopsshop.com/index.php?...ndex&cPath=7_8
Product Listing page is OK - http://www.gopopsshop.com/index.php?...&products_id=7

My problem is related to Colorbox because pages were OK before Colorbox.

I am unable to find where the size of the images are controlled.
On the Categorys page, the pix are sized to 98 x 80 px - skinny and tall - wrong orientation
On the Products page, the pix are sized to 147 x 120 px - looks good

What is overriding the code on the Category page that causes the picture size to change so dramatically????

Looking forward to suggestions

I don't "see" the issue that you are referring to.. Will say this.. Colorbox doesn't set or modify image dimensions.. Sugget you look at your Configuration > Images settings..

[duluthpopsshop]

I don't "see" the issue that you are referring to.. Will say this.. Colorbox doesn't set or modify image dimensions.. Sugget you look at your Configuration > Images settings..

My bad completely - have been so used to working with Google Chrome that I forgot to check the site with Firefox. After your post, I checked the site in Firefox and there are no problems. Will have to look into the Chrome issue.

Thanks for responding so quickly.

Fred

[pricediscrimination]

Was using Zen Lightbox before. It stopped working after upgrading to 1.5.4, so installed Zen Colorbox.

One feature that was nice in Zen Lightbox was the option to "Close on Image Click". It is possible to do this with Zen Colorbox?

Also, not sure if these are known bugs or just bugs pertaining to my specific installation, but setting "Display Image Counter" to false causes the Colorbox to stop working.

[pricediscrimination]

Almost forgot, the colorbox is resizing images to be smaller than they actually are. The point of a lightbox is to show images that are larger than being displayed on the page. What is the best way to go about making the images display as their actual size?