Closed (fixed)
Project:
Scrollable Content
Version:
6.x-2.0-beta2
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
4 Feb 2011 at 15:45 UTC
Updated:
19 May 2011 at 21:21 UTC
Here is first draft of ported module to Drupal 7
Works for me at first look
| Comment | File | Size | Author |
|---|---|---|---|
| scrollable_content-7.x-2.x.patch | 6.92 KB | Andrey Zakharov |
Comments
Comment #1
good_man commentedThanks a lot for your work, I didn't manage to get it into cvs version (what tag you run the patch against?), note the spaces and tabs and follow Drupal coding standards, also you generated the patch using `cvs diff` or `diff` ? better use `cvs diff -up > patch_name.patch`.
no need for this anymore, D7 already uses jQuery 1.4
should put the config url.
extra indentation spaces.
also extra indentation spaces.
we use two spaces in Drupal Coding Standards.
tab, use spaces in your editor to replace tabs.
extra spaces
use spaces instead of tab.
no images cache anymore in D7, it's image style and included in core.
spaces
lots of tabs, and better replace inline javascript with Drupal.behaviors.settings array
tab
tab
Powered by Dreditor.
Comment #2
good_man commentedPorted and now in dev. version, report bugs in separate issue(s).