CVS edit link for bennpearson

I want to submit a theme based on baselinecss - http://baselinecss.com/
Tim Millwood is trying to turn me into a themer as I work for Mark Boulton. I have watched all the Lullabot tutorials and think I'm ready with Tim's help to submit a theme. Thanks
CommentFileSizeAuthor
#8 baselinecss.zip16.26 KBAnonymous (not verified)
#6 baselinecss.zip16.14 KBAnonymous (not verified)
#1 baselinecss.zip27.78 KBAnonymous (not verified)

Comments

Anonymous’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new27.78 KB

Attached is the current version of the theme although I will add the baseline grid JavaScript as it looks here: http://baselinecss.com/

timmillwood’s picture

Note: I have been working with Benn on all our latest projects to get him up to speed with Drupal theming. I second his CVS application.

avpaderno’s picture

Status: Needs review » Needs work

Files available from third-party sites should not be included in Drupal.org CVS.
Files that are not compatible with GPL License V2+ are not acceptable in Drupal.org CVS.

Anonymous’s picture

Status: Needs work » Needs review

I could commit it without the baseline files and state in the documentation that the files would need to be downloaded.

avpaderno’s picture

Status: Needs review » Needs work

That is fine.

No new code has been uploaded.

Anonymous’s picture

Status: Needs work » Needs review
StatusFileSize
new16.14 KB

Thanks, here are the new files.

avpaderno’s picture

Status: Needs review » Needs work

jquery.autofill.js is available from third-party sites.
Then, why do the JavaScript files contain // $Id: jquery.autofill.js,v 1.1.4.1 2009/08/05 18:55:54 add1sun Exp $, and // $Id: javascript.js,v 1.1.4.2 2009/08/12 23:29:56 add1sun Exp $?

The file readme.txt should be renamed README.txt.

Anonymous’s picture

Status: Needs work » Needs review
StatusFileSize
new16.26 KB

I have removed the JavaScript files, added ; $Id$ to the .info file and renamed the README.text

avpaderno’s picture

Status: Needs review » Fixed

The directories images, and styles need to contain at least one file (you can put a file README.txt to explain what they are for); differently, who will download the archive file created by the packaging script will not have those directories.

avpaderno’s picture

Issue tags: +Theme review

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

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

avpaderno’s picture

Component: Miscellaneous » new project application
Assigned: Unassigned » avpaderno
Issue summary: View changes
Status: Closed (fixed) » Fixed

Status: Fixed » Closed (fixed)

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