CVS edit link for markku

I would like to submit a theme I created based on an older version of Clean theme.
I call it Green Clean and it is based off of a theme I created for http://www.haliburtonfresh.com
I've also included a light-weight, fast loading version of the theme as well called Green Clean Lean
Screenshots are at http://eclipsemediaco.net/files/greenclean.jpg and http://eclipsemediaco.net/files/greencleanlean.jpg (eclipsemediaco.net is my website)
Currently the theme is for Drupal 5.x and 6.x and I'd like to create it for 7.x.
I've been a front-end web developer for 10 years, and have been using Drupal for over 5 years, and think I'm over-due for contributing to the community.
I have expert HTML, XHTML, CSS and PHP skills and would be an excellent addition to the Drupal community.
CommentFileSizeAuthor
#7 greenclean-6x.zip260.42 KBstongo
#1 greenclean-6x.zip266.98 KBstongo
#1 greenclean-5x.zip177.59 KBstongo

Comments

stongo’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new177.59 KB
new266.98 KB

Green Clean is a tableless layout with a left sidebar built on a "Clean" theme base.
It's compatible with IE6+, Firefox, Safari and Chrome.
The Drupal 6.x version comes with two sub-themes, Green Clean Light & Green Clean Lean. Green Clean Light uses a smaller background image for those concerned about load time. Green Clean Lean is stripped down and super fast loading.
All themes contain the following block areas: left, before content, content, header, footer, footer message.

avpaderno’s picture

Issue tags: +Theme review
avpaderno’s picture

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

Hello, and thanks for applying for a CVS account. We just review a module / theme per applicant; let us know which one you want to be reviewed, and let us know.

stongo’s picture

The Drupal 6 module is fine.

Thanks

avpaderno’s picture

Status: Postponed (maintainer needs more info) » Needs review
avpaderno’s picture

Status: Needs review » Fixed
  1.   <?php print $head ?>
    

    There should always be the semicolon at the end (except in <?php if ($test): ?>).

  2. The license file should not be included, as it is not possible to commit it in Drupal.org repository.

Thank you for your contribution! I am going to update your account.
These are some recommended readings to help with excellent maintainership:

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

I thank all the dedicated reviewers as well.

stongo’s picture

StatusFileSize
new260.42 KB

Ok, here is a revised theme using your feedback.
Thanks!

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