11 posts
  • Bought between 1 and 9 items
  • Exclusive Author
  • Has been a member for 2-3 years
  • Sold between 10 000 and 50 000 dollars
okathemes says

hi, there? please help me improve my theme as i have uploaded it second time and make all possible changes the reviewer wants, but they still say it not upto there mark. Advice me what i need to improve in it,

Live Demo: http://www.greepit.com/atif/neoarts/index.html

Thank
Atif

1531 posts
  • Bought between 10 and 49 items
  • Envato Studio (Microlancer) Beta Tester
  • Exclusive Author
  • Has been a member for 3-4 years
webdesignerart says

1) Please add padding on bottom where text(Main, ABout Us etc etc) 2) Slider not such cool, and any fade simple transitions. 3) Our work bullet navigation not looks good to me. And mouseover on thumbnail the description comes up please make speed slow or add fade in transition. 4) Contact form is not such crisp plz make some very crisp look now its very boldy, Submit button looks cool :). 5) Background changing animation its too fast pls slow the speed and attractive background thy are not so cool. 6) Typography its not good “Welcome to the best in modern design” the font is very normal plz search the attractive simple fonts. 7)

$(function()
            {
                // this initialises the demo scollpanes on the page.
                $('#pane1').jScrollPane();
                $('#pane2').jScrollPane();
                $('#pane3').jScrollPane();
                $('#pane4').jScrollPane();
            });

Use only 1 class name instead of different different IDs 8) Very Bad practise :(
<script type="text/javascript" src="js/jquery.min.js"></script>
<script type="text/javascript" src="js/jquery-1.7.1.min.js"></script>
<script type="text/javascript" src="js/jquery-1.4.4.min.js"></script>
<script type="text/javascript" src="js/jquery.mousewheel.js"></script>
<script type="text/javascript" src="js/jScrollPane.js"></script>

why you use two jquery versions Use only latest one. follow this structure:
<script type="text/javascript" src="js/jquery-1.7.1.min.js"></script>
<script type="text/javascript" src="js/jquery.mousewheel.js"></script>
<script type="text/javascript" src="js/jScrollPane.js"></script>

Question? why u use gradiant white in content area please try just white color.

Suggestion: Your border radius its not looks good please reduce its radius.

When u design VCard template so please consider the template have no any Vertical Scrollbar because it also give not finished look.

I liked :) 1) About, Main, Our Services etc etc mouse over orange fadein in effect. 2)

<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />

=========================================================

So now after these above changes will apply then Inshallah your theme will approve :)

11 posts
  • Bought between 1 and 9 items
  • Exclusive Author
  • Has been a member for 2-3 years
  • Sold between 10 000 and 50 000 dollars
okathemes says

Thanks for helping me, i will do the changes accordingly as you told me. it was great help. could you tell me how much padding i should put in the main headings. like Main, about us etc?

by
by
by
by
by
by