Html 3rd soft reject, iam lost, help requested


#1

Hello everyone, iam trying to get my first item (Html template) approved on Theme Forest, however, iam currently facing some difficulties, so it would be great if someone with experience could help me point out the issues to fix in order to get this template approved. And i also think i need a better explanation of what Envato reviewer consider as spacing and alignment issues. Here is my review history :

NEW ITEM SUBMISSION

Thank you for your submission. 1. PLACEHOLDER IMAGES: Please delete the images from the main file and replace them with placeholders. http://placehold.it - Please mention this in the item description so that buyers understand photos are not included in the download. 2. PROPER EVENT BINDING: Consider using the preferred .on() method rather than .click(), .bind(), .hover(), etc. For best performance and concise code use event delegation whenever possible: http://code.tutsplus.com/tutorials/quick-tip-javascript-event-delegation-in-4-minutes--net-8961 3. STRICT MODE REQUIRED: All JavaScript should be written with “use strict” mode on. Please note that strict mode is scoped. For concise code it is recommened to placed 4. CSS TABLE OF CONTENTS: Make sure the CSS file is well documented with proper table of contents. For more information, see: http://www.smashingmagazine.com/2008/05/02/improving-code-readability-with-css-styleguides/ 5. SCROLLING ANIMATIONS: The scrolling animations are currently overused. Animation is a tool that should be used sparingly to help enforce the concept of your design. Please consider where you are using animations and their purpose.

FIRST RESUBMISSION

  1. It’s recommended that you pair complementary fonts that add to the design aesthetic you’re going for. In general, a design should utilize less than three (ideally two) font variations that each compliment another — not compete for attention. Ref: https://designschool.canva.com/blog/the-ultimate-guide-to-font-pairing/ Example(s): https://envato.d.pr/D4ZA6v 2. The choice of typeface should support the purpose of the design and by extension, the needs of the intended audience. Ref: https://designschool.canva.com/font-design/ 3. There are spacing and alignment issues in this item. Please make sure that all elements are aligned properly and spacing is consistent from section to section. Here are some articles with more information regarding negative space and proper spacing: http://webdesign.tutsplus.com/articles/using-white-space-or-negative-space-in-your-designs–webdesign-3401, http://webdesign.tutsplus.com/articles/adding-space-to-your-designs–webdesign-14836 example: https://envato.d.pr/GAb8lY and more throughout. 4. Inline CSS is not permitted. Please remove all inline CSS.

SECOND RESUBMISSION

THE CHANGES ARE NOT ENOUGH: Unfortunately, the changes are not enough and the previous review still stands. Please carefully check your item to make sure all problems were adequately addressed before resubmitting this item.

Iam kinda lost, because i tried to address all issues the best i could. But however, it seems there are still problems without furthermore details from the reviewer (fmontes). I really want to get my item approved, but i don’t know where to start at the moment. I think that iam not understanding TF requirements about spacing (everything has to be even ? how to judge if there is too much or not enough), so please explain to me what i didnt get.

Here is the demo link for the template :
https://bulkit.cssninja.io

Feedback really appreciated, thanks for your time


#2

Don’t be lost. I’ve got like ~30 soft rejects for the ShopKit theme, maybe even more! Just be patient and follow the guides reviewer has for you. It might even be a path you must take, not like fix this and that. So, when you do check all of the above + all the next things that might occur, then you’ll get approved! So, patience and persistence and you’ll get there! Thanks!


#3

Your advice is appreciated mate, thank you ! it gives me a little hope :smiley: Iam reworking all spacing from the ground to make it even. I guess they dont like elements with padding only on one side :smiley:


#4

image

  1. Change this font

  1. Here, make better contrast and change this font too to the same font as you’ll use for the “Hello.I am Bulkit.” text, and maybe, just maybe see why the padding isn’t right there, but that’s just an opinion, rest are the facts

image

  1. Here bad resolution and a bit boring, I’d add one more “line/chart” in the mixed color with some opacity and do try SVG for these as you would benefit a lot

I’d say that’d be it and you’d be approved. The font that should be used for the 1. and 2. should be nice, fresh an boldy, so do search for “font typography trends 2018” :wink:

Thanks!


#5

image

  1. And if you want it to be perfect do something about these too. I’m not sure this is it the size or the opacity that’s bothering me, but probably you can do a lot better here too.

#6

Thank you very much for taking the time to review my work. I am now reworking spacing for the ground and i will implement all of your remarks. I think maybe i need to change the gradient and its opacity, that may be what is bothering you


#7

It’s just an opinion about the animation, maybe I’d like them smaller and less opaque if any, not sure how’d that look. But overall design wise, colors and the rest looks good! So, you’re not that far away :slight_smile:


#8

Thank you very much, your feedback is much appreciated, if you don’t mind, i will send you a link in a few days, when iam finished updatiing the template ! Kudos to you @dzeriho


#9

Hey there, it took me a big week to make the required changes, here is a link to the updated version, wich i have just submitted :

https://bulkit.cssninja.io

Dont forget to ctrl + F5 to get all the changes !


#10

Nice your themes and I hope good sales for sell :slight_smile:


#11

I hope it gets approved :slight_smile:


#12

Quick note :slight_smile: the contrast issue in the headings is still there. It’s not just about your screen, people have all sorts of display they use, if it looks good on yours, doesn’t mean it’ll not look like an error elsewhere. New font is just fine, but as said, if the contrast in the headings was a bit more, the font would’ve looked better. One more thing, logo should have used the same font style, this way you still got 2 fonts which don’t go together. The logo in the sticky header shouldn’t be cyan at all, it should flow with the rest of the design. And if you want to make that first appearance stand out, then this icon is also a bit weird.
image
Animation down there, that was mentioned before still looks the same :slight_smile: but we’ve agreed that’s just an opinion, maybe majority will like it the way it is. Thanks!


#13

Thanks for the update mate, maybe I didn’t get what you mean by constrast, is it about increasing the color dephth by making it darker to make headings stand out more ?

I will try to see if I can improve the animation overall render :smile:

And for the icon, it looks good on my screen but seems to have margin issues on yours, I’ll check this out. Thanks for the feedback mate !


#14

Yes, the contrast in my opinion was with the 1,2… numbers, they look on one of the screens almost white. Thought that was obvious :slight_smile: No worries!


#15

Yeah you are right, maybe the numbers opacity is too low, I’ll look into it and test on other screens


#16

Got rejected again, i guess i don’t understand envato spacing requirements, does everything have to be perfectly even (padding-top = padding-bottom etc)… ?

Here is the reviewer comment :

  1. Typography still needs improvement in some areas. example: https://envato.d.pr/Fu0N04 and more. 2. Still an issue. There are a few inconsistent spacing/padding as well as alignment issues throughout the design. Please go through your files and weed out these basic design issues.

I would be really grateful if someone could point out those spacing and alignment issues, because at that point , i can’t see them myself. @dzeriho pointed out the hamburger icon, but i could really use some guidance for the other spacing issues.

Quite hard to become an envato author, but i won’t give up, i just need to understand what they want, because i invested more than 30 hours fixing issues since the last review to get again this type of reviewer comment, without any details about the issues to fix. It’s quite frustrating.

demo link
https://bulkit.cssninja.io

Thank you for your help


#17

Anyone ?


#18

Well, it’s pretty obvious that the less important stuff (in this case category) has more weight to it than the title, which should be the main focus? Don’t be at all frustrated, just do your best every time and that one time when everything fits its place will be the time you get approved. Next item will be far easier. Thanks!


#19

I am getting these 2 issues. Would you please give me suggestions-

  1. SPACING AND ALIGNMENT: There are spacing and alignment issues in this item. Please make sure that all elements are aligned properly and spacing is consistent from section to section. Here are some articles with more information regarding negative space and proper spacing:

  2. STRICT MODE REQUIRED: All JavaScript should be written with “use strict” mode on. Please note that strict mode is scoped. For concise code it is recommended to placed on top of the Javascript file.

@dzeriho