I am a Designer and Webworker from Germany. I will upload my themes for the drupal community. This is my first theme to upload(demo site): http://www.reizkraft.com
best regards
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | bluefun.zip | 69.01 KB | tom78 |
| #4 | bluefun.zip | 68.94 KB | tom78 |
| #1 | bluefun.zip | 67.33 KB | tom78 |
| #1 | blfun.jpg | 58.41 KB | tom78 |
Comments
Comment #1
tom78 commentedThe zip file and a screenshot. It's a simple theme with 2 lavalamp menus, 1-3 columns and a fixed width.
Demo: http://reizkraft.com
Comment #2
ultimateboy commentedThe theme is good, minor details only. The javascript included in the template.php should simply be placed in the .info file instead. The javascript in page.tpl should, but doesnt really have to be put into its own .js file and included via .info. Doing it this way will include it in the javascript compression.
I'm not so sure I like this being in the page.tpl. Is there not an alternative using the classes that drupal provides? If not, I'd probably use hook_preprocess_page() to add an additional body class based on those conditions, and then use css to set the properties needed.
There is also quite a few empty divs in your page.tpl (mainly in the header). Remember that you have the "clear-block" class available, so you dont have to add empty clear divs:
<div class="clr"></div>.Finally, a minor thing is "Powered by Reizkraft.com" in the footer... I dont mind "designed by" or "drupal theme by" but "powered" seems a bit strong.
Really none of these are game stoppers, just small details.
Comment #3
tom78 commentedthanks... :)
ok. i will change it...
best regards
Comment #4
tom78 commentedNow it is ok... :)
best regards
Comment #5
tom78 commentedups. Sorry... this is the file.
Comment #6
ajk commentedComment #8
avpaderno