Bug Found with move and loop option after 12 hours running

herdiana's Avatar

herdiana

25 Oct, 2012 04:14 AM

Hi,
I have tested RoyalSlider with following option below:
- loop : true, - transitionType : 'move' - fullscreen -> enabled:true

then I saw bug in RoyalSlider after running about 12 hours, I think the bug is come from CSS position. I also inspect the element with firebug and I saw the left position is about 2.3505e+7px. I think the position value is to big, so the slider cannot work. It would be great If you can fix the bug, because I have to use this slider with FullScreen feature until 12 hours. Find attachment to view the screenshots and files.

Thanks for your support.

  1. herdiana closed this discussion on 25 Oct, 2012 04:15 AM.

  2. herdiana re-opened this discussion on 25 Oct, 2012 04:16 AM

  3. Support Staff 1 Posted by Dmitry Semenov on 25 Oct, 2012 07:37 AM

    Dmitry Semenov's Avatar

    Hi,

    When loop is enabled slider doesn't add/remove looped slide, it just reorders it without touching current slide or position of whole container to avoid flickering. This causes continuous growing of main container position.

    Max position that browsers support is about 135 000 000 pixels. So if width of your slide is 960px, you get about 140 000 slide changes before it breaks.

    This even proves your statement about 12h of work with autoPlay 300ms.

    3.3 moves/sec * 60sec * 60min * 12h = 142 560 slide changes
    

    So how do you fix it.
    At first reduce number of moves per second by increasing autoplay dealay to 1000ms, this will increase working-time to 36h.

    Secondly, add counter that will refresh page every X number of slides. e.g.:

    var slideWidth = 900,
          maxX = 135000000,
          slider = $('.royalSlider').data('royalSlider');
          maxNumOfMoves = maxX / slideWidth * 0.8; // -20% to be safe
    
    slider.ev.on('rsBeforeMove', function() {
        if(slider.currSlideId > maxNumOfMoves) {
            location.reload();
        }
    });
    

    Hope this solution fits your needs. If you tell me what for do you need it to slide all the time for 12h, I might find some more apporpriate solution for you.

    I don't want to add such checks to core slider to not to overload it, as such issues are extremely rare.

    Dmitry

  4. Support Staff 2 Posted by Dmitry Semenov on 25 Oct, 2012 07:39 AM

    Dmitry Semenov's Avatar

    [edit comment above]

  5. 3 Posted by herdiana on 25 Oct, 2012 11:08 AM

    herdiana's Avatar

    Dear Dmitry,

    Thank you for your reply and thorough explanation.

    I'm using RoyalSlider to display live tweets on a TV screen. The tweets
    will be displayed one tweet at a time, so one tweet one slide. I'm using
    appendSlide for new tweets, and removeSlide to manage max. number of
    slides. Great feature in RoyalSlider!

    So ideally I'd like to have it run for 12 to 24 hours a day without any
    hitch and 0 user interaction. That's why I previously also asked about
    running automatically in native full-screen.

    Please let me know if you think of a more appropriate solution.

    Thanks so much!

Reply to this discussion

Internal reply

Formatting help / Preview (switch to plain text) No formatting (switch to Markdown)

Attaching KB article:

»

Already uploaded files

  • slide_loop_fullscreen.jpg 185 KB
  • slide_loop_nofullscreen.jpg 262 KB
  • loop-move-fs.zip 3.07 MB

Attached Files

You can attach files up to 10MB

If you don't have an account yet, we need to confirm you're human and not a machine trying to post spam.

Keyboard shortcuts

Generic

? Show this help
ESC Blurs the current field

Comment Form

r Focus the comment reply box
^ + ↩ Submit the comment

You can use Command ⌘ instead of Control ^ on Mac