Spectrum

Hi heresh,
good theme, I proceed with a manual review:
1) in your .info file you include files

stylesheets[all][] = system-menus.css
stylesheets[all][] = system-menus-rtl.css
stylesheets[all][] = style.css
 

but I can't find system-menus.css and system-menus-rtl.css
2) Consider adding a style-rtl.css file or an alternative style for LT languages (you can read more here). The included style.css seems to be a RTL stylesheet, I provide a screenshot.
3) Adding a

scripts[] = js/jquery-min.js

you are adding a jQuery v1.3.2 on top of the default Drupal 6 jQuery version, without replacing it. It could break some modules.
4) It seems you are working on git master branch, you should switch to a 6.x-1.x branch.
5) In template.php there is a

t('نویسنده: <strong> !username </strong>'

consider write in only english language, to let translator localize better every sentence.

I add also an automatic review with some code style issue, you can read it here:
http://ventral.org/pareview/httpgitdrupalorgsandboxheresh1325968git

Greetz

Hi heresh,
sorry for the broken link, the correct link is here:
http://drupal.org/node/1127732

About jquery upgrading, I suggest you to use the noConflict() function, you can read more here:
http://drupal.org/node/1058168
remember to document any "not ordinary" implementation like this inside your theme to make easier for reviewers to review your project.

Cheers

what's going on with your demo-site?

as there are currently many applications in queue we need more reviewers, especially reviewers for themes are rare!
so think about getting a review bonus and we will come back to your application sooner.

Hi heresh,
you're totally right about the way you use to override system-menu css.
I'm sorry, this was my mistake, you can read that your implementation is the right one here (the text in red color): http://drupal.org/node/263967
I'm very sorry about that, I thought you had to create also an empty css file inside your theme directory once you specified it on .info file, but this is not true.

you should remove it then from your project page (it's the "try out a demonstration link" in the sidebar)

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