753 posts
  • Has referred 10+ members
  • Has sold $10,000+ on Envato Market
  • Has collected 1+ items on Envato Market
  • Had an item featured on Envato Market
+4 more
AdamGold says

Hi, my theme has been rejected for no reason (Jarel didn’t give any reasons). What do you think:

http://tview.co.il/themeforest/jd/

It supposed to be a very minimal yet informational portfolio, what should I improve? Thanks.

315 posts
  • Has referred 50+ members
  • Has sold $250,000+ on Envato Market
  • Has collected 50+ items on Envato Market
  • Sells items exclusively on Envato Market
+6 more
bitpub says

Hello.

I like the concept but the idea could be refined further.

Changes I would do:
  1. Make sure the document is valid xhtml 1.0 transitional. I checked it with http://validator.w3.org/ and it has 7 reviews.
  2. Total width: I have 1280×1024 resolution and it barely fits on my screen. Maybe you should change proportions and make it fit nicely in 1024px
  3. Html source code
    • format it nicer
    • make it semantic html
    • stuff like using < br > instead of wrapping text in < p > elements is a big no no
    • add a favicon and metadata in the head
    • don’t link to the YUI REset but copy it in your css for faster load times
    • maybe 5 columns would be enough in the work gallery
    • for the section titles I wouldn’t go with that double overlaping text. Keep it simple. Also use font replacement for it not images
  4. In the css:
    • improve the typography – vertical rhythm is important, you have no line-height and font-sizes and proportions are non existent. Use a traditional size scale: 12, 14, 16,18, 20, 22, 24px
    • text colors are not balanced – the grey it to light and black to strong
    • the same for the navigation colors
  5. Add a working contact form
  6. I know that Ie6 is not required to be supported but how can you ignore so much market share – it would be nice to get to work in it as well
675 posts
  • Has referred 10+ members
  • Has sold $10,000+ on Envato Market
  • Has collected 10+ items on Envato Market
  • Sells items exclusively on Envato Market
+6 more
dexterous says

In my opinion:

I Don’t think using images for the text links (the main navigation) is good idea. It should be text.

I see horizontal scroll bar (never good)

Though the image you have used is good, but the whole thing is missing something, I cannot explain, but it appears that I can scroll the page down (when in fact I cannot) .. do you see what I mean?? I mean really i kept trying to scroll down or up .. it is missing some borders or boxes may be ..

Well these are just my thoughts..

as bitpub said, overall idea is good.

2022 posts Review Manager
  • Has referred 10+ members
  • Has sold $5,000+ on Envato Market
  • Has collected 50+ items on Envato Market
  • Member of the Envato Team
+12 more
jremick Envato team says
Hi, my theme has been rejected for no reason (Jarel didn’t give any reasons). What do you think:

http://tview.co.il/themeforest/jd/

It supposed to be a very minimal yet informational portfolio, what should I improve? Thanks.

Hey Adam, that’s not quite true. I give reasons with every review. You’re welcome to reply to the review message and I’d be more than happy to answer your questions via support ticket.

753 posts
  • Has referred 10+ members
  • Has sold $10,000+ on Envato Market
  • Has collected 1+ items on Envato Market
  • Had an item featured on Envato Market
+4 more
AdamGold says

@bitpub -

Make sure the document is valid xhtml 1.0 transitional. I checked it with http://validator.w3.org/ and it has 7 reviews. – Done.

Total width: I have 1280×1024 resolution and it barely fits on my screen. Maybe you should change proportions and make it fit nicely in 1024px – What do you mean? How do you see it?

Html source code -

format it nicer – Will do.

make it semantic html – what do you mean?

stuff like using < br > instead of wrapping text in < p > elements is a big no no – Done. (changed the < br />)

add a favicon and metadata in the head – Will do.

maybe 5 columns would be enough in the work gallery – Done

for the section titles I wouldn’t go with that double overlaping text. Keep it simple. Also use font replacement for it not images – Actually I kinda like it. Don’t you think I can keep it like it is now?

Add a working contact form – Will do.

I know that Ie6 is not required to be supported but how can you ignore so much market share – it would be nice to get to work in it as well – I will try.

Thank you so much!

@dexterous -

I Don’t think using images for the text links (the main navigation) is good idea. It should be text. – It is text replacement, it’s not images. (FLIR)

I see horizontal scroll bar (never good) – What size is your screen and what resulotion do you use?

Thanks!

@Jarel -

Here is your message:

“Thanks so much for taking the time to submit this template. Unfortunately, due to our high standards, your template has been declined.

ThemeForest requires a well designed template with valid code and proper documentation for approval. This template will need significant improvements in one or more of these areas before it will be ready for ThemeForest. You might consider taking a look at some of our most popular items to get a better idea for what we’re looking for. You can view our popular items here: http://ThemeForest.net/page/top_sellers

You might consider submitting the template to http://www.reviewmydesign.net to get input and ideas from other great designers and ThemeForest authors.”

I really appriciate your support, you’re great.

315 posts
  • Has referred 50+ members
  • Has sold $250,000+ on Envato Market
  • Has collected 50+ items on Envato Market
  • Sells items exclusively on Envato Market
+6 more
bitpub says

For a contact form you could try – http://www.bitrepository.com/a-simple-ajax-contact-form-with-php-validation.html

It needs more work but it is a good start :) I’ll come back with round 2 of feedback soon.

753 posts
  • Has referred 10+ members
  • Has sold $10,000+ on Envato Market
  • Has collected 1+ items on Envato Market
  • Had an item featured on Envato Market
+4 more
AdamGold says

I added an ajax contact form ! (not the one you gave, something else :D) Thanks!

What else?

315 posts
  • Has referred 50+ members
  • Has sold $250,000+ on Envato Market
  • Has collected 50+ items on Envato Market
  • Sells items exclusively on Envato Market
+6 more
bitpub says

I say you still need to work on the contrast, typography and also make it fit on 1024 px resolution. I made a layout that I think achieves all of these:

http://img694.imageshack.us/img694/9596/contrastjpg.jpg

If you want to use these ideas to modify your template feel free to do so. I made the file just for you, out of fun :D. Contact me through my user page if you want the layered .png version

753 posts
  • Has referred 10+ members
  • Has sold $10,000+ on Envato Market
  • Has collected 1+ items on Envato Market
  • Had an item featured on Envato Market
+4 more
AdamGold says

Thank you so much!

http://tview.co.il/themeforest/jd/

A big improvment. Is it better? Thanks.

1036 posts
  • Has referred 200+ members
  • Has sold $125,000+ on Envato Market
  • Has collected 50+ items on Envato Market
  • Elite Author: Sold more than $75,000 on Envato Market
+9 more
Pixelworkshop says

Hi Adam,

My humble opinion : the typography could be improved a lot, the repeated “lorem ipsum” doesn’t look good to me. I would also recommend you to use a grid (like the 960gs), it helps a lot to create the PSD and the coding part.

Try also to resize your browser, you’ll see that the content is hidden under the navigation, as bitpub said it should fit in a 1024 screen (the grid helps for that :)

Good luck

by
by
by
by
by
by