[Plugin: Image horizontal reel scroll slideshow] Improvement/Bug fixes

Image horizontal reel scroll slideshow (Ihrss) is a great start to a plugin, but with a few bugs that had yet to be sorted. A bit of an inconvenience so I’ve coded them out.

We are running a site for a charity expedition (https://www.ChasingTheSunrise.org) and as such on a tight budget. Grateful as ever to the open source world of WP ??

So one for the knowledge base until Gopi gets around to sorting this out…

Problem 1: element images dont resize correctly – if you want to have a scroller 100px high then every uploaded image needs to be 100px high. Worst case is if they’re bigger than your scroller height then you only see the very tops of the images.

Fix: In image-horizontal-reel-scroll-slideshow/image-horizontal-reel-scroll-slideshow.php

Two mods need to be done twice for complete fix, 2 mods in function “Ihrss_shortcode” and 2 mods in function “Ihrss()” – one is for the shortcode, one is for the widget.

In the if ( ! empty($data) ) {} condition, first add the opening line:
$Ihrss_sliderelementwidth = $Ihrss_sliderheight * 4/3; // This line is added to set 4/3 image aspect ratio based on the slider height as a reference dimension.

Next up the line:
$Ihrss_package = $Ihrss_package .”IHRSS_SLIDESRARRAY[$cnt]=’<img alt=\”$Ihrss_title\” src=\”$Ihrss_path\” />‘; “;

Can be amended with a style attribute in the <img> tag:
$Ihrss_package = $Ihrss_package .”IHRSS_SLIDESRARRAY[$cnt]=’<img alt=\”$Ihrss_title\” src=\”$Ihrss_path\” style=\”max-height:”.$Ihrss_sliderheight.”px; max-width:”.$Ihrss_sliderelementwidth.”px; vertical-align:middle;\” />‘; “;

Problem 2: v.11.8 doesnt have, or seems to have lost, support for the Display=NO image attribute

Fix: In image-horizontal-reel-scroll-slideshow/image-horizontal-reel-scroll-slideshow.php

In line:
$sSql = “select Ihrss_path,Ihrss_link,Ihrss_target,Ihrss_title from “.WP_Ihrss_TABLE.” where 1=1″;

change the superflous “1=1” condition to “Ihrss_status=’YES'”:
$sSql = “select Ihrss_path,Ihrss_link,Ihrss_target,Ihrss_title from “.WP_Ihrss_TABLE.” where Ihrss_status=’YES'”;

Problem 3: more a mod for better functionality to allow 100% width to fit with theme styles

Fix: In image-horizontal-reel-scroll-slideshow/image-horizontal-reel-scroll-slideshow.js
Change text ‘+IHRSS_HEIGHT+’ (incl. apostrophies) to a 100% term. Three occurances, either:
Two of “width:100%;” in div tags
One of “width=100%” in ilayer tag

Hope this helps folks out and that Gopi makes use of our debugging (the comments section of the support page is closed so its on the forum instead)