Zen Colorbox

[DivaVocals]

Would it not be prudent to add Zen Colorbox to the IH4 readme by name and vice versa? I am not sure it would have helped me with my issue. But I am sure that I will not be the only one to miss this important connection.

Thanks
Chris

Sure.. and if you look at the date Zen Colorbox was submitted, and the last IH4 update, you'd know that Zen Colorbox was submitted AFTER the last IH4 update..

Submitting an IH4 update JUST to add this to the readme wasn't IMHO a compelling reason to submit and update.. With the recent v1.5.4 release I will be submitting an IH4 update and I WILL update the readme to INCLUDE Zen Colorbox, and also to suggest that folks NOT install Fual Slimbox OR Zen Lightbox anymore in favor of this module..

[dw08gm]

Not part of the core code.. Images only.. There is a BETA version in the authors Github repo which is in response to this issue: https://github.com/daniel-hopkins/zen-colorbox/issues/9.

Be advised that the code is UNTESTED and INCOMPLETE, and there is NO SUPPORT for it here really. In order to have continuity across a shop using this module, the plan was to address ALL the popups in the list. 3 remain outstanding, and the author has been busy and has not been able to do any further work on this feature. ANYONE who wants this feature fully functioning to fork his repo and pick up where he left off on this feature would be a hero! SHARE IT with the community!

I will be submitting an update to Zen Colorbox soon. The updated code is in my Zen Colorbox Github fork of the Zen Colorbox repo.. I submitted a pull request with my modifications to the master repo, but because the author has been busy he has not yet approved and incorporated my changes into his master repo.. So just know that IF you decide to install this version with this code and you also want the LATEST and greatest version, there will be discrepancies between my fork and the author's repo so you will have to merge them together..

Thanks for the headsup. I will look into it over the next few days.

[lat9]

FWIW, I just downloaded the Colorbox repository and installed it on a Zen Cart v1.5.4 demo site without issue.

[DivaVocals]

FWIW, I just downloaded the Colorbox repository and installed it on a Zen Cart v1.5.4 demo site without issue.

Cool.. BUT there's a caveat.. Can you test some of the popups to make sure they work.. when I tested the code from the main repo, Daniel had started working on code to make the popups work inside a lightbox.. (See: https://github.com/daniel-hopkins/zen-colorbox/issues/9) It's not ALL popus, but the issue link I posted tells you which ones.. I tested it BRIEFLY, and got some errors (details of which escape me.. I just rolled back the popup code..) Let me know if the popups are working for you..

[lat9]

Is there a setting that causes the colorbox interface to be used. I'm not seeing the colorbox-type popup shopping-cart help.

[DivaVocals]

Is there a setting that causes the colorbox interface to be used. I'm not seeing the colorbox-type popup shopping-cart help.

Hmmmmmmmmmmmmmmmmmmm.. To be honest I don't remember.. I didn't realize that Daniel had committed the popup code to the repo (Github did not notify me) until I went to grab the latest from his repo for my latest client install.. I installed tested, failed, and then rolled back.. Decided to go back at testing when I wasn't on a deadline..

[lat9]

Now I've got myself confused! I "think" that I downloaded the master from your fork rather than from the danielhopkins account.

Certainly looking at the danielhopkins account, there are changes for the shopping_cart page that aren't reflected in the version I downloaded. I'll get the danielhopkins "master" installed and let you know ...

[DivaVocals]

Now I've got myself confused! I "think" that I downloaded the master from your fork rather than from the danielhopkins account.

Certainly looking at the danielhopkins account, there are changes for the shopping_cart page that aren't reflected in the version I downloaded. I'll get the danielhopkins "master" installed and let you know ...

That's because I didn't include Daniel's popup changes in my fork.. I plan to submit my fork this weekend.. Daniel just accepted the changes from my pull request in his repo, and they will be included i nmy update.. I am submitting my fork because Daniels popup changes are incomplete (3 popups are not "done" yet) and untested..

[lat9]

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.

[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.

Awesome.. Will include the validation change (thought I caught that) and submit this weekend..