In its current state I’d say no but its a work in progress so I cant really judge it on that.
I’m thinking that ThemeForest will start to tighten up how many of these type of themes get accepted on here because they all function the same so you need to add something to give it more of a unique selling point.
Impact is a header font so you shouldn’t really be using it for links.
I just think it doesn’t have anything special at the moment unfortunately but its a good start.
Your headers, nor your “Our Work” heading don’t really stand out from the body text and if you’re going to use icons you really should do it for all navigation items or none.
Your logo would also need some work, but as the buyer would change this I couldn’t understand a theme being rejected for the logo!
I like your header background! Keep working and tweaking it!
I will use icons for all the navigation items.
It still looks quite plain and flat, try to add some texture, subtle gradients or depth to it.
I think now you’ve got to many fonts, either stay all Serif or all Sans Serif fonts else it will look out of place.
As I said before you really shouldn’t be using Impact for the menu links, maybe use it for the headings where you currently have the serif font, that would work I think.
It’s getting there slowly