This small module provide the jquery.marquee.js for any node / block /view.
Demo: http://www.moovedev.be/moove/
Very simple to use, just add

<marquee behavior="scroll" direction="up" height="75" scrollamount="1" width="150"><p>Your text</p></marquee></p>

in your node's body or block.

It works with views and token:
add a field in your view.
then add this code (change the token_value) in the view's header:

<marquee behavior="scroll" direction="up" scrollamount="1"
          height="65" width="290">
          <p>[token_value]</p>
     </marquee>

http://drupalcode.org/sandbox/Moove/1579648.git

CommentFileSizeAuthor
#2 drupalcs-result.txt3.59 KBtargoo
jquery_marquee_js.zip3.4 KBjbfelix

Comments

targoo’s picture

targoo commented
Status: Needs review » Needs work

Welcome !

It appears you are working in the "master" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.
Review of the master branch:

This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. Get a review bonus and we will come back to your application sooner.

Source: http://ventral.org/pareview - PAReview.sh online service

targoo’s picture

targoo commented
StatusFileSize
new drupalcs-result.txt3.59 KB

here is the attachment ;-)

patrickd’s picture

patrickd commented

As installation and usage instructions are quite important for us to review, please take a moment to make your project page follow the tips for a great project page.

We do really need more hands in the application queue and highly recommend to get a review bonus so we can come back to your application sooner.

jbfelix’s picture

jbfelix commented
Status: Needs work » Needs review

Moved to 7.X-1.x branch.
Documentation added and code cleaned.

sylvain lecoy’s picture

sylvain lecoy commented
Status: Needs review » Needs work

Manual review:

While this module seems very simple to test, I see some blockers to me:

1) You should use a hook_node() / _block() / _view() to add you js file.
a) either you want to effectively put it on all the pages. Use hook_boot() instead.
b) there is no need to use hook_html_head_alter() if you are not making use of this hook.

I put it as Need work.

MattA’s picture

MattA commented

If you're going to add JS to every page you can do so in the module's .info file. See Managing JavaScript in Drupal 7 for more information.

Beyond that there doesn't seem to be much code to check (note checklist section 2.3).

gazoakley’s picture

gazoakley commented
Status: Needs work » Closed (won't fix)
Idle Application
This thread has been idle, in the 'needs work' state with no activity for several months. Therefore, I am assuming that you are no longer pursuing this application, and marking it as closed (won't fix).

If this is incorrect, and you are still pursuing this application, then please feel free to re-open the thread and set the issue status to "needs work" or "needs review", depending on the current status of your code.