Soft Rejected - Need Feedback to rectify the issues

Envato Authors CodeCanyon
1

My HTML template was soft rejected recently, Here are the reasons given by reviewer:

  1. Parts of your design are either difficult to read or have contrast issues. Please make sure all sections of your design have adequate contrast and all text is easily readable.

  2. http://envato.d.pr/IWMP (I don’t actually understood this objection as I’m already using ‘use strict’ in my custom JS file)

Now, plz suggest which area I should improve in the context of above points.

Item URL: http://html.mstrends.com/_bare/index.html

UPDATE: I’ve added some screenshots on redpen.io to ease the reviews:

Homepage: https://redpen.io/fe0e2c4c8001c6383f

Blog: https://redpen.io/wk85f420b92d1a9ef5

Blog Single: https://redpen.io/b1a591ee-8988-4be2-8c67-3d12941087fd

Shop: https://redpen.io/ta0c6fa7662f0ce181

Shop Single: https://redpen.io/jo077262d7428e777e

Shop Cart: https://redpen.io/bf653fc7-a2d7-4d92-803f-c7c2578a4456

Update # 2:

For readability, I’ve experimented by changing the body and sub-heading font on some one’s suggestion. Theses are screenshots of homepage, applied different fonts:

Old Typography: http://d.pr/i/1hilC

New Typography: http://d.pr/i/OPaC

To which would you vote?

2

The “use strict” directive is new in JavaScript, It is not a statement, but a literal expression, ignored by earlier versions of JavaScript.
The purpose of “use strict” is to indicate that the code should be executed in “strict mode”.
With strict mode, you can not, for example, use undeclared variables.
In strict mood using a variable (property or object) without declaring it, is not allowed.
Simply add “use strict” in the beginning of your js codes and be aware about declaring variables before call them in functions.

1 Like
3

Thanks for the reply.

That’s what I don’t understand. I’ve actually added “use strict” at the beginning of my JS code and this screenshot is from my own code. No explanation was provided for this screenshot.

4

I suggest to replace cursor icon with hand icon and for pressed icon simply you can use hand grabbed icon.
https://www.google.com/search?client=opera&q=hand+icon&sourceid=opera&ie=UTF-8&oe=UTF-8
Add some font-weight to subtitles in “Work” area. It’s hard to read.
Change border-color of text fields and textarea, I think dark colors will be better.
I suggest to change font-color in “Learn More” button. Maybe White is ok in active mood.

1 Like
5

Strict mode makes several changes to normal JavaScript semantics. First, strict mode eliminates some JavaScript silent errors by changing them to throw errors. Second, strict mode fixes mistakes that make it difficult for JavaScript engines to perform optimizations: strict mode code can sometimes be made to run faster than identical code that’s not strict mode. Third, strict mode prohibits some syntax likely to be defined in future versions of ECMAScript.

6

Thanks again for the suggestion.

I agree sub-heading and fields border could be a bit darker and I’ll see what I can do with button text color.

Regarding hand icon, are you referring to slider cursor icon. If so, you are right it was supposed to be hand icon, it seems to be a glitch.

7

Yes, I forgot to transparent my meaning :slight_smile: , Slider Cursor Icon.

1 Like
8

For readability, I’ve experimented by changing the body and sub-heading font on some one’s suggestion. Theses are screenshots of homepage, applied different fonts:

Old Typography: http://d.pr/i/1hilC

New Typography: http://d.pr/i/OPaC

To which would you vote?

9

I vote for the new one.
Just checked the old with font-weight: 300 instead of 400 for p element and it is also looks better.

By the way, the contrast issues should be with the buttons (learn more links in old design), when I set font weight: 500, it seems to be sharper.

10

Please remove the custom scroll functionality, it’s horrible UX

2 Likes
11

Thanks for you input. Paragraph text in ‘About’ section will look good if it is a bit lighter but setting font-weight: 300 site-wide will cause readability issues, because it would be too thin (provided we use old font). Although, font-weight: 300 would do fine with new typography (Roboto font)

I’m not sure about font-wieght: 500 for button text, as its font doesn’t have this weight on my site.

12

Thanks for you suggestion, I’ve tested the site in desktop (with mouse) and on laptop (with trackpad) and have not encountered any such issue. Can anybody confirm if it’s happening to them. Is it any specific device related issue?

13

On Windows it may be cool, but on MAC it’s horrible.

1 Like
14

Thanks for the heads up, I’ll check it on mac then.