Results 1 to 5 of 5

Thread: Crosstick script has problems in Firefox, help?

  1. #1
    Join Date
    Apr 2005
    Posts
    2
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Cool Crosstick script has problems in Firefox, help?

    Script: DD Crosstick
    http://www.dynamicdrive.com/dynamicindex2/crosstick.htm

    I copied the code exactly and the script works marvelously on IE, but it only shows the top two items in Firefox and then goes blank and never cycles back again. Any suggestions?

    It is a WONDERFUL script.

    I have it implemented on my work-in-progress page at http://www.thelightradio.net/index.php

    Thanks,
    matthew

  2. #2
    Join Date
    Mar 2005
    Location
    SE PA USA
    Posts
    30,495
    Thanks
    82
    Thanked 3,449 Times in 3,410 Posts
    Blog Entries
    12

    Default

    Either take this:
    Code:
    <?xml version="1.0" encoding="iso-8859-1"?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
    off the page or figure out why it is causing the problem. I vote for taking it off.
    - John
    ________________________

    Show Additional Thanks: International Rescue Committee - Donate or: The Ocean Conservancy - Donate or: PayPal - Donate

  3. #3
    Join Date
    Mar 2005
    Location
    SE PA USA
    Posts
    30,495
    Thanks
    82
    Thanked 3,449 Times in 3,410 Posts
    Blog Entries
    12

    Default

    Figured it out, you can keep that declaration if you change these two lines in the script:
    Code:
    tdiv.style.top=parseInt(scrollerheight)
    change to:
    Code:
    tdiv.style.top=parseInt(scrollerheight)+"px"
    and change:
    Code:
    tdiv2.style.top=parseInt(scrollerheight)
    to:
    Code:
    tdiv2.style.top=parseInt(scrollerheight)+"px"
    - John
    ________________________

    Show Additional Thanks: International Rescue Committee - Donate or: The Ocean Conservancy - Donate or: PayPal - Donate

  4. #4
    Join Date
    Mar 2005
    Location
    SE PA USA
    Posts
    30,495
    Thanks
    82
    Thanked 3,449 Times in 3,410 Posts
    Blog Entries
    12

    Default

    I just posted the above as a bug in bug reports because IMHO it is a bug. While doing so I realized that adding +"px" is redundant. You could as easily remove the parseInt() from them and accomplish the same thing. So, they'd end up looking like this:
    Code:
    tdiv.style.top=scrollerheight
    and this:
    Code:
    tdiv2.style.top=scrollerheight
    respectively. This is actually preferable.
    - John
    ________________________

    Show Additional Thanks: International Rescue Committee - Donate or: The Ocean Conservancy - Donate or: PayPal - Donate

  5. #5
    Join Date
    Apr 2005
    Posts
    2
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default Thank you!

    Thank you for your help! That worked marvelously!

    You're my new hero!

    -matthew

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •