my template was rejected :(

Envato Authors Theme Authors
,
1

Hi, I uploaded a template but they rejected it :frowning: Could you help me with feedback to find out what I did wrong?

here is the link of my template

thank you everybody

2

hi @mamolinav, Iā€™m working now so Iā€™ll just go on some modification and someone else can give you the rest:

  1. Donā€™t use CDN at all
  2. Make a CSS components table to give a list of what you did in that file.
  3. donā€™t write the script in the HTML page ( Iā€™m not sure this if can be counted as a disadvantage ).
  4. you need more content and sections ( this one is important if the template will help the buyer ).
    to the icons in the navbar.
  5. the effects on the button are not good if the page is simply the elements should be simple.
  6. your sections and content are not good at all because there is some unusual section without any reason, plus youā€™re not delivering a project to one category as I can see u included more than just one.
  7. I can see pixels of your images, resize them to fit your boxes, and update your formā€™s design.
  8. your breadcrumb design doesnā€™t meet the requirements.
  9. you have a javascript error in your script.js file on line 170 or 175 I guess.

your template can go great you just have to make it look modern and clean and it will get approved.

1 Like
3

Hi @mamolinav

I will make some points that I know
Not all points but the points that I understand
I like you oldest template and was rejected every time :joy:

use ā€˜google chrome extensions page rulerā€™

  • https://prnt.sc/q3jhup

  • PROPER EVENT BINDING: Consider using the preferred .on() method rather than .click(), .bind(), .hover(), .load(), .ready(), etc.

@OMINIMO
Make a CSS components table to give a list of what you did in that file.
Make a js components table to give a list of what you did in that file.

@OMINIMO
you have a javascript error in your script.js file on line 170 or 175 I guess.

Uncaught TypeError: Cannot read property ā€˜minā€™ of null
at init2slider (script.js:174)
at script.js:163

I hope to have another one offers you help usefully

2 Likes
4

@themearabia you got this man :star_struck:

5

ohh thanks, i didnā€™t realize :0

6

just one question, what do you mean when you say ā€œmake a css and js components tableā€ ? i donā€™t understand how to make it.

Another thing, so do i have to put icons in the navbar for each section?

thank you so much :smiley:

7

Table of contents
css ex
https://themes.themearabia.net/html/basma-resume-html/one_page/ltr/style.css
look: TABLE OF CONTENTS

Table of contents
js ex
https://themes.themearabia.net/html/basma-resume-html/one_page/ltr/js/custom.js
look: Table of contents

I donā€™t understand what you mean

8

Oh great!! I agree with you, it would be more ordered.

the other question is related to what OMINIMO said:

" You need more content and sections ( this one is important if the template will help the buyer ).
to the icons in the navbar."

I donā€™t get it :frowning: