Your design has less flaws in terms of spacing and alignment although there are still some minor issues, like spacing between titles and icons, porfolio overlay caption vertical alignment etc … However it is not the main point.
Your template cruelly lacks of aesthetics regarding today standards. I mean, look at some approved templates and you will see that you are still miles away from being accepted with this kind of item. Typography is really weak, wether it is the fonts (typeface) or the typography hierarchy. The color scheme is so outdated. You really need to use flashy colors (but the right way) like modern designers do today.
Your demos are clones of the same page with different headers, even multipage is the same page sliced into multiple short pages. This will not be enough in terms of features. You’ve also chosen an overcrowded niche. There are litterally thousands of portfolios templates on TF and, honestly, even if you were approved with this design, you wouldn’t make sales. Some free creative templates still do the job better than yours.
I think it is a time for reflexion, where you need to think about learning real design. You can code a decent template, but ThemeForest is about something else, it is about design. And remember it is a shop forum, not a webdesign forum. Take time to learn more about design, take the time to think about your next product, and take time to craft it.
If you can’t get good at design, at least ask a PSD author to share rights with you, and start converting already cleanly designed templates. Keep working hard and you will get there.