Great work! I've tested scrolltext in IE and firefox, and it really scrolls in a cross-platform way.

Essential improvement - For up and down scrolltexts, It's better to have new line between each title to the next. Here is a quick patch which does it - I've added '
' after each link for up and down scrolltexts. It's much better then the current situation.

Amnon
-
Professional: Drupal Israel | Drupal Development & Consulting | Eco-Healing | Effective Hosting Strategies | בניית אתרים
Personal: Hitech Dolphin: Regain Simple Joy :)

CommentFileSizeAuthor
scrolltext.module.patch331 bytesdruvision

Comments

druvision’s picture

druvision commented

Typo correction: I've added '<br>' after each link for up and down scrolltexts.

drupalnesia’s picture

drupalnesia commented
Status: Reviewed & tested by the community » Closed (fixed)

Sorry for late answer. Since every text using l() function then no need to add new line because l() automatic add new line between every title.
If you need more space berween line then you cau modify the DIV id="scrolltext" from your css file.