Project Page and Git repository
http://drupal.org/sandbox/eleonel/1683916

git clone --recursive --branch master eleonel@git.drupal.org:sandbox/eleonel/1683916.git xurxo
cd xurxo

Version

Drupal version: 7x

Screenshot

Xurxo ThemeReviews of other projects

http://drupal.org/node/1683092#comment-6233262
http://drupal.org/node/1682682#comment-6233248
http://drupal.org/node/1591818#comment-6033430

Files: 
CommentFileSizeAuthor
#8 footer.jpg4.88 KBketelagoreng
#8 sidebar.jpg8.14 KBketelagoreng

Comments

Welcome!

You have to set the status to "needs review" if you want to get a review. You have listed some reviews in your issue summary, but you did not do any manual reviews? Please read through the source code of other projects and report any issues that you find. And don't forget to add the review bonus tag if you did some actual manual reviews as outlined in #1410826: [META] Review bonus.

Status:Active» Needs review

Status:Needs review» Needs work

Manual review:
In styles/block.css
line number 6: there should be space after ":"
line number 9: there should be space after ":"
line number 25, 31: use lowercase in hexcolor codes.
suggestion: try using #fff hex color code instead of white
line number 46: css properties should be in alphabetical order.

Same way in style.css: use lowercase in hexcolor codes. at line number 572, 462, 458
in style.css there are spacing issues.

Please take a moment to make your project page follow tips for a great project page.

You are working in the "master" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.

Done.

Thanks!

Status:Needs work» Needs review

StatusFileSize
new8.14 KB
new4.88 KB

hi eleonel,

It's a nice theme, but I got some problems here. Please looks some picts I attached.
pict 1: unreadable title post at footer
pict 2: it's not a bad idea to give some padding

Status:Needs review» Needs work

Status:Needs work» Needs review

hi, thanks for your feedback. Fixed!

Issue summary:View changes

Adding screenshot section :)

Status:Needs review» Needs work

Hi eleonel,

Manual review:
1) Please upload screenshot in this page
2) add menu active style and you can give menu divider

#navigation a.active {
    background: none repeat scroll 0 0 #FFFFFF;
    color: #E34212;
}
#navigation li{
   border-right: 1px solid #CCC;
   /*-- you can choose any other border color ---*/
}

3) LHS block not floated ( screenshot attached)

.sidebar .gutter .block{
    float: left;
    width: 100%;
}

Thanks,
Shanid kv

Status:Needs work» Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

If you reopen this please keep in mind that we are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)

Status:Closed (won't fix)» Needs review

Thanks. Fixed!

Status:Needs review» Needs work

There are some errors reported by automated review tools, did you already check them? See http://ventral.org/pareview/httpgitdrupalorgsandboxeleonel1683916git

I'm a robot and this is an automated message from Project Applications Scraper.

Status:Needs work» Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.

Issue summary:View changes

Resize screenshot image ;)