valquiria23 Posted June 23, 2018 Share Posted June 23, 2018 Dear Zahid @JcMagpie How can I hide the 3 circles of the carousel in this addon? Simple Carousel header for Edge https://apps.oscommerce.com/xa3f9&simple-carousel-header-for-edge I need to hide them because if I put on the image text in that area those 3 horizintales circles get in the way.... Best regards Valqui Quote Community Oscommerce fan You'll find the latest osC community version here. Link to comment Share on other sites More sharing options...
♥JcMagpie Posted June 24, 2018 Share Posted June 24, 2018 (edited) 10 hours ago, valquiria23 said: Dear Zahid @JcMagpie How can I hide the 3 circles of the carousel in this addon? Simple Carousel header for Edge https://apps.oscommerce.com/xa3f9&simple-carousel-header-for-edge I need to hide them because if I put on the image text in that area those 3 horizintales circles get in the way.... Best regards Valqui Simply edit------- public_html/includes/modules/content/header/templates/jccarousel.php and comment out the following code. <!--<ol class="carousel-indicators"> <li data-target="#myCarousel" data-slide-to="0" class="active"></li> <li data-target="#myCarousel" data-slide-to="1"></li> <li data-target="#myCarousel" data-slide-to="2"></li> </ol>--> tested on my site and works fine. They do look better without them. https://tbyb.co.uk/ Edited June 24, 2018 by JcMagpie valquiria23 1 Quote Link to comment Share on other sites More sharing options...
valquiria23 Posted June 26, 2018 Author Share Posted June 26, 2018 thank you very much for your attention Zahid !! I made the changes and it worked correctly. Thank you very much for your contributions to the community! Greeting Valqui Quote Community Oscommerce fan You'll find the latest osC community version here. Link to comment Share on other sites More sharing options...
♥raiwa Posted December 14, 2018 Share Posted December 14, 2018 Hello Zahid @JcMagpie, It would be better to use tep_image functions for the image calls. Quote About Me: http://www.oscommerce.com/forums/user/249059-raiwa/ Need help? How To Get The Help You Need Is your version of osC up to date? You'll find the latest osC community version CE Phoenix here. Public Phoenix Change Log Cheat Set on Google Sheets Link to comment Share on other sites More sharing options...
♥JcMagpie Posted December 14, 2018 Share Posted December 14, 2018 (edited) Thank you @raiwa for your advice . As others will maybe need more info on this perhaps you could explain the advantage of this over the simple html that I used? I simply used this which most people will understand if they need to make changes. <img src="images/<?php echo MODULE_CONTENT_HEADER_JCCAROUSEL_IMAGE1?>" alt="Image one"> The temp function is also quite simple. tep_image(path_of_image, alt_tag_of_image, width_of_image, height_of_image); which would change the code to somthing like this, <?php require('includes/application_top.php'); echo tep_image('images/' . 'MODULE_CONTENT_HEADER_JCCAROUSEL_IMAGE1', "Image one"); ?> Would be interesting to know the benefits gained on a simple 3 line add-on like this? As always happy to learn and improve. Edited December 14, 2018 by JcMagpie Quote Link to comment Share on other sites More sharing options...
♥raiwa Posted December 14, 2018 Share Posted December 14, 2018 - compatibility with add-ons using the tep_image function to auto generate thumbnails like KissIt image thumbnailer - ensure images are responsive (responsive image class is auto added) - following oscommerce coding guidelines - auto include title and alt from the second parameter - shows nothing instead of broken image pic if image is not available Another big improvement would be to use the built in banner functionality for the images. Quote About Me: http://www.oscommerce.com/forums/user/249059-raiwa/ Need help? How To Get The Help You Need Is your version of osC up to date? You'll find the latest osC community version CE Phoenix here. Public Phoenix Change Log Cheat Set on Google Sheets Link to comment Share on other sites More sharing options...
♥JcMagpie Posted December 14, 2018 Share Posted December 14, 2018 I'm sure you are correct. Thank you. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.