Make WordPress Themes

Change History (3)

#1 @scui2
10 years ago

  • Owner set to scui2
  • Status changed from new to reviewing

#2 @scui2
10 years ago

The ticket will not be approved due the following required issue:

  • Missing text domanin in style.css and incorrect text domain used throughout the. The correct text domain should be winds
  • PHP Error for Link post. Please note this still happens after I activated required plugin and maintained meta box.
  • Please wrap $content_width in a function - not in top level of functions.php.
  • All global variables need to be prefixed e.g: enqueue.php link 362-363
  • Please use standard WordPress logo and favicon function
  • Copyright text is not escaped during output. Footer.ogo link 39
  • The theme seems to use OffCanvas for primary sidebar. But there is no navigation to open the offcanvas sidebar
  • The Theme is implementing LightBox for images. As the results, Gallery images are not displayed.
  • Social Links in customizer are not allowed as they are considered as Content Creation
  • Please use theme_mod instead of option. See below link:

https://make.wordpress.org/themes/2014/11/26/customizer-theme-mods-api-or-settings-api/

  • Declare the licenses for all resources (images, fonts, js, css) used in the readme.txt
  • Non presention meta boxes are not allowed in theme: e.g. title URL, embed URL. I understand you defines meta box for portfolio post type also. But it seems that there are not used in the theme.
  • Deprecated tags in style.css: white, light and gray
  • Some texts in sidebar.php are not translatable. Maybe more. Also you should not point to wp-admin/widgets.php unless the logged on user is an admin.
  • Please explain why you define windstheme_comments function in comments.php and why you need line 44-46.

Please visit for the latest requirements, test and review your theme against the latest requirement and upload again.

https://make.wordpress.org/themes/handbook/review/required/

I will keep the ticket open for 7 days. Please let me know if you can fix the required issues.

#3 @scui2
10 years ago

  • Resolution set to not-approved
  • Status changed from reviewing to closed

As part of the new theme directory guidelines, reviewers are now allowed to close tickets if a theme has 5 or more issues. You can read more about this change here: http://make.wordpress.org/themes. This is our effort to reduce the queue time so theme authors will not have to wait for 5 to 6 months.

Please don't get discouraged. When a ticket is closed, it's not the end. We want you to resubmit your theme. We just ask that you spend a bit more time making sure that your theme meets all the guidelines before we make it live.

If you need help with anything or want me to explain further, don't hesitate to post here in this ticket. I'm happy to answer any questions you have. Or, feel free to post over on our Slack channel http://make.wordpress.org/themes."

Although all the issues listed in my previous comments are required, I am closing this ticket due to the following major issues.

  • Missing text domanin in style.css and incorrect text domain used throughout the theme. The correct text domain should be winds in your case.
  • PHP Error for Link post. Please note this still happens after I activated required plugin and maintained meta box.
  • All global variables need to be prefixed e.g: enqueue.php link 362-363
  • Copyright text is not escaped during output. Footer.php link 39
  • The Theme is implementing LightBox for images. As the results, Gallery images are not displayed correctly.
  • Social Links in customizer are not allowed as they are considered as Content Creation
  • Please use theme_mod instead of option. If you See below link:

https://make.wordpress.org/themes/2014/11/26/customizer-theme-mods-api-or-settings-api/

  • Non presentation meta boxes are not allowed in theme: e.g. title URL, embed URL.
  • Some texts in sidebar.php are not translatable. Maybe more. Also you should not point to wp-admin/widgets.php unless the logged on user is an admin.
Note: See TracTickets for help on using tickets.