That reject has nothing to do with code quality though, it’s about design, in this case you have been given a very clear sense of direction, make it perfect.
Minimal templates can be clean and simple when you manage to handle the white space and details to provide a simple but yet unique template. But if you can’t, it will look like an incomplete template – blank template.
In this case, the submitted template is more like a ‘framework’ than a full fledged design. If you’re going with minimalism, you need to make sure all the little details check out.
Personally, it completely depends on the attitude of the customer. I’ve exchanged 20+ emails with some customers because they were pleasant to help.
+1 – If someone is nice, friendly, and doesn’t expect instant ‘my client wants this’ help then I’ll always go the extra mile, every time.
Opening a portfolio item, the scroll goes to the top of the loaded container, but is overlapped by the header(nav)
Envato could tie buyer rewards into badges, could offer marketplace credit for building up buyer badges, i.e “Got the 500-999 badge? Here’s $50 marketplace credit”
Save theme weight and use the built in Customisation API – https://codex.wordpress.org/Theme_Customization_API – honestly, I think it should be standards to start removing external options panels now, the built in API can do everything an external options panel can do without adding all that extra theme weight & potential security flaws.
We’re all still a little confused by the new requirements, but so far I’ve had no problem with the .js used exactly as my JSFiddle above, but still, I’ve flagged this for staff reply to see if we can get a little clarificiation