Does the slide show affect hits?

ballentphotoballentphoto Registered Users Posts: 312 Major grins
edited August 16, 2006 in SmugMug Pro Sales Support
On my main page www.ballentphoto.com I have the slide show running. I just got the slideshow running today and my conversion ratio is uhmmm 796% :scratch with the large image spiked at 5000K+ hits I have only been on for a few days so I am not sure what is going on here... is someone slurping up pics or something else is happening ???? My header jpg is only showing 18 hits so anyone have an idea? :dunno

Thanks much,
Michael
-Michael
Just take the picture :):
Pictures are at available at:http://www.ballentphoto.com

My Blog: http://ballentphoto.blogspot.com

Comments

  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 15, 2006
    Do the hits correlate to the gallery being used for the slideshow ?

    But yes, I am pretty sure that any images being displayed will be included as hits in statistics.
    David Parry
    SmugMug API Developer
    My Photos
  • ballentphotoballentphoto Registered Users Posts: 312 Major grins
    edited August 15, 2006
    devbobo wrote:
    Do the hits correlate to the gallery being used for the slideshow ?

    But yes, I am pretty sure that any images being displayed will be included as hits in statistics.

    Yes the gallery that is "pegged" is the one that I am using for the slideshow... I am a newbie to this so I am not sure what is going on. But that does make sense... the hit stats look great for only being online for a few days, hehehehe just odd the way it's reporting it... of course I could have goofed the javascript stuff.
    -Michael
    Just take the picture :):
    Pictures are at available at:http://www.ballentphoto.com

    My Blog: http://ballentphoto.blogspot.com
  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 15, 2006
    ok i think I have found the problem, your intent is to only display the slideshow on your homepage...right ?

    If so, your slideshow is loading on every page not just your homepage.

    To fix this, change this code in your footer...
    loadSlideshow(); 
    

    to this...
    if (YD.hasClass(document.body, "homepage"))
      loadSlideshow(); 
    
    David Parry
    SmugMug API Developer
    My Photos
  • brandofamilybrandofamily Registered Users Posts: 2,013 Major grins
    edited August 15, 2006
    Will that code, instead of using display:none and then the display:block for the homepage allow pages to load quicker? Or is your way JS and the other the equivalent in CSS?
  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 15, 2006
    Will that code, instead of using display:none and then the display:block for the homepage allow pages to load quicker? Or is your way JS and the other the equivalent in CSS?

    Anthony,

    there is a difference. CSS code turns on/off the visibility for the slideshow container.

    But calling or not calling that function is the difference of whether the slideshow images are loaded or not...does that make sense ?

    A good way to determine if you are acutally calling the slideshow when you should be, is to remove any CSS that hides it. Then browse through your site, if the slideshow appears on any page (not including the base image if used) then you are downloading extra stuff to the clients browser that isn't required and actually slowing down their experience.
    David Parry
    SmugMug API Developer
    My Photos
  • ballentphotoballentphoto Registered Users Posts: 312 Major grins
    edited August 15, 2006
    devbobo wrote:
    ok i think I have found the problem, your intent is to only display the slideshow on your homepage...right ?

    If so, your slideshow is loading on every page not just your homepage.

    To fix this, change this code in your footer...
    loadSlideshow(); 
    
    to this...
    if (YD.hasClass(document.body, "homepage"))
      loadSlideshow(); 
    
    I made the change, thanks for looking at it. I took the code from the instructions, guess that what you get when you are dealing with hacks hehehe. That would explain why the galleries took longer to load.

    You ROCK!!!
    -Michael
    Just take the picture :):
    Pictures are at available at:http://www.ballentphoto.com

    My Blog: http://ballentphoto.blogspot.com
  • brandofamilybrandofamily Registered Users Posts: 2,013 Major grins
    edited August 15, 2006
    OK. I'm certain that the slide show is loading... as you may remember, I have the main slideshow on the homepage and another on my "gear" page. I also have a "virtual" featured gallery...

    Here's the code from my footer...
    <!-- Common Slideshow Parms -->
    <*****script type="text/javascript">
    slideshowContainerId = "brandoSS";
    quickStart = false;
    slideshowType = 0;
    slideDuration = 3;
    showSlideTransition = true;
    randomSlides = true;
    slideControls = false;
    
    if (isClass("homepage") && !isClass("featured"))
    { /*== slideshow parms specific to homepage slideshow ==*/
      slideshowUrl = "http://www.brandolinoimaging.com/gallery/1760778";//album to use for slideshow photos
      resizeToPhoto = true;
      firstSlideUrl = "http://brandolinoimaging.smugmug.com/photos/87313324-M.jpg";
      slidesClickable = false;
      slideHeight = "450"; //visible height of slide
      slideWidth = "577";  //visible width of slide
      photoHeight = "450"; //actual photo height
      photoWidth = "577"; //actual photo width
    }
    
    else if (isClass("gallery_1091885") && isClass("galleryPage"))
    { /*== slideshow parms specific to 1091885 slideshow ==*/
      slideshowUrl = "http://brandolinoimaging.smugmug.com/gallery/1085730"; //album to use for slideshow photos
      firstSlideUrl = "http://brandolinoimaging.smugmug.com/photos/54160348-L.jpg";
      resizeToPhoto = false;
      slideHeight = "350"; //visible height of slide
      slideWidth = "350";  //visible width of slide
      photoHeight = "350"; //actual photo height
      photoWidth = "350"; //actual photo width 
    }
    
    loadSlideshow();
    <*****/********
    <!-- End of Common Slideshow Parms -->
    

    Any thoughts on loading correctly for MY site...
  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 15, 2006
    I made the change, thanks for looking at it. I took the code from the instructions, guess that what you get when you are dealing with hacks hehehe. That would explain why the galleries took longer to load.

    You ROCK!!!

    no problem.

    I am going to talk to BWG about adding a failsafe to prevent this condition.
    David Parry
    SmugMug API Developer
    My Photos
  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 15, 2006
    only call it when u want it :D

    change to this...
    <!-- Common Slideshow Parms -->
    <*****script type="text/javascript">
    slideshowContainerId = "brandoSS";
    quickStart = false;
    slideshowType = 0;
    slideDuration = 3;
    showSlideTransition = true;
    randomSlides = true;
    slideControls = false;
    
    if (isClass("homepage") && !isClass("featured"))
    { /*== slideshow parms specific to homepage slideshow ==*/
      slideshowUrl = "http://www.brandolinoimaging.com/gallery/1760778";//album to use for slideshow photos
      resizeToPhoto = true;
      firstSlideUrl = "http://brandolinoimaging.smugmug.com/photos/87313324-M.jpg";
      slidesClickable = false;
      slideHeight = "450"; //visible height of slide
      slideWidth = "577";  //visible width of slide
      photoHeight = "450"; //actual photo height
      photoWidth = "577"; //actual photo width
      [B][COLOR=Yellow]loadSlideshow();[/COLOR][/B]
    }
    
    else if (isClass("gallery_1091885") && isClass("galleryPage"))
    { /*== slideshow parms specific to 1091885 slideshow ==*/
      slideshowUrl = "http://brandolinoimaging.smugmug.com/gallery/1085730"; //album to use for slideshow photos
      firstSlideUrl = "http://brandolinoimaging.smugmug.com/photos/54160348-L.jpg";
      resizeToPhoto = false;
      slideHeight = "350"; //visible height of slide
      slideWidth = "350";  //visible width of slide
      photoHeight = "350"; //actual photo height
      photoWidth = "350"; //actual photo width 
      [B][COLOR=Yellow]loadSlideshow();[/COLOR][/B]
    }
    <*****/********
    <!-- End of Common Slideshow Parms -->
    
    David Parry
    SmugMug API Developer
    My Photos
  • brandofamilybrandofamily Registered Users Posts: 2,013 Major grins
    edited August 15, 2006
    OK.. I think I get it... I had programming many years ago... the 2 sections are setting the parameters for loading... the 1st says load if condition A is true but condition B is false.
    The 2nd says load if BOTH C AND D are true... am I close??
  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 15, 2006
    OK.. I think I get it... I had programming many years ago... the 2 sections are setting the parameters for loading... the 1st says load if condition A is true but condition B is false.
    The 2nd says load if BOTH C AND D are true... am I close??

    nod.gif
    David Parry
    SmugMug API Developer
    My Photos
  • brandofamilybrandofamily Registered Users Posts: 2,013 Major grins
    edited August 15, 2006
    devbobo wrote:
    nod.gif
    thanks I'll fix my code and remove the CSS code...thumb.gif
  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 15, 2006
    thanks I'll fix my code and remove the CSS code...thumb.gif

    leave the CSS code there, I only mentioned it as a method to determine if u are incorrectly loading the slideshow
    David Parry
    SmugMug API Developer
    My Photos
  • ballentphotoballentphoto Registered Users Posts: 312 Major grins
    edited August 15, 2006
    devbobo wrote:
    no problem.

    I am going to talk to BWG about adding a failsafe to prevent this condition.
    The hits still seem to be growing... is this because the pages are still cached on people's machines?headscratch.gif

    Below is my footer code.
    type="text/javascript">
    /* required slideshow variables */
    slideshowContainerId = "featuredBox"; 
    slideshowUrl = "http://www.ballentphoto.com/gallery/1759024"; 
    /* optional slideshow variables */ 
    /* see properties section */ 
    if (YD.hasClass(document.body, "homepage"))
    loadSlideshow();
    
    -Michael
    Just take the picture :):
    Pictures are at available at:http://www.ballentphoto.com

    My Blog: http://ballentphoto.blogspot.com
  • brandofamilybrandofamily Registered Users Posts: 2,013 Major grins
    edited August 16, 2006
    devbobo wrote:
    leave the CSS code there, I only mentioned it as a method to determine if u are incorrectly loading the slideshow
    If I'm not loading the show on other pages do I really need to HIDE it for those pages?
  • bwgbwg Registered Users, Retired Mod Posts: 2,119 SmugMug Employee
    edited August 16, 2006
    devbobo wrote:
    only call it when u want it :D

    change to this...

    meh, i'm not a big fan of calling the same function twice for no reason. that code you gave him will behave no differently than his original code.

    the slideshow will not execute if:
    • slideshowContainerId is not provided
    • slideshowContainerId does not evaluate to a valid element
    • slideshowUrl is not provided
    in brando's case, unless one of his conditions were met, slideshowUrl was not set and calling loadSlideshow(); would not have done anything.

    failsafe enough?
    Pedal faster
  • brandofamilybrandofamily Registered Users Posts: 2,013 Major grins
    edited August 16, 2006
    bigwebguy wrote:
    meh, i'm not a big fan of calling the same function twice for no reason. that code you gave him will behave no differently than his original code.

    the slideshow will not execute if:
    • slideshowContainerId is not provided
    • slideshowContainerId does not evaluate to a valid element
    • slideshowUrl is not provided
    in brando's case, unless one of his conditions were met, slideshowUrl was not set and calling loadSlideshow(); would not have done anything.

    failsafe enough?
    Thanks BWG. I'll leave my code alone then... Thanks too to Dev for looking...
  • devbobodevbobo Registered Users, Retired Mod Posts: 4,339 SmugMug Employee
    edited August 16, 2006
    bigwebguy wrote:
    failsafe enough?
    no, cuz this situation can still happen as did in the original thread. Can u add an additional condition that it doesn't loadSlideshow if the slideshowcontainer.style.display='none'
    David Parry
    SmugMug API Developer
    My Photos
Sign In or Register to comment.