CVS edit link for e-anima

Hi,

archive:http://www.artwaves.de/1_ARTWAVES/views_horizontal_slider.zip

-- General --
i am working with drupal since i started playing around with drupal 5. Over the years i started learing the API and first doing some custom modifications on some modules.
A few month ago i started digging deeper (reading Drupal Pro Module Development) and started to write my own modules from scratch because i needed some functionality that was not given yet.

-- The Module --
The attached archive contains a Module (views style plugin) i recently wrote for a project im working on. I attached a sample page of the page i wrote it for.
There are a lot of slider/fading modules on drupal.org that fade single images with a lot of various animations. There is also a vertical accordion Module.

As you can see in the sample, my module emulates some kind of horizontal image/content gliding with light weight jquery.

-- Why i want to commit --

I think this Module could be useful for others that want to have such a sliding effect. Befor starting to code this module i thought, that it would be faster for me when i only use a normal list view
output and put my jquery code into the page header. But then i thought about it again and decided that this is not a very re-usable approach, and so i wrote this style plugin.

I used a lot of modules over the years and want to contribute something myself.

-- Abou Myself --

Userprofile on drupal.org: e-anima http://drupal.org/user/255202
Irc freenode: e-anima #drupal.de, #drupal ..

greets

Marco

Comments

marcoka’s picture

Status: Needs work » Postponed (maintainer needs more info)
Issue tags: -Module review
StatusFileSize
new5.02 KB
dawehner’s picture

Status: Postponed (maintainer needs more info) » Needs review

.

avpaderno’s picture

Status: Needs review » Needs work
Issue tags: +Module review

Hello, and thanks for applying for a CVS account. I am adding the review tags, and some volunteers will review the code, pointing out what it needs to be changed.

As per requirements, the motivation message should be expanded to contain more features of the proposed project. For themes, it should include also a screenshot of the theme, and (when possible) a link to a working demo site using the proposed theme; for modules, it should include also a comparison with the existing solutions.

marcoka’s picture

Status: Postponed (maintainer needs more info) » Needs work
Issue tags: +Module review
StatusFileSize
new5 KB

Code corrections.
Setting up a demo-site, will be avaliable here soon: http://drupal-demo.artwaves.de/

dawehner’s picture

Status: Needs work » Reviewed & tested by the community

I looked at the module and talked with him over irc.

There was some code style errors and some security issues. They got fixed in #4 so

There is no comparable module yet, i think.

gerhard killesreiter’s picture

Status: Reviewed & tested by the community » Fixed

Granted CVS access

Status: Fixed » Closed (fixed)
Issue tags: -Module review

Automatically closed -- issue fixed for 2 weeks with no activity.

avpaderno’s picture

Component: Miscellaneous » new project application
Issue summary: View changes