Make WordPress Themes

Change History (3)

#1 @acosmin
6 years ago

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

#2 follow-up: @acosmin
6 years ago

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

Hi @lightwp
I have had a quick look at the theme and at this point, it does not follow the requirements.

The Theme Review Team does not hard reject themes, -we want you to resubmit your theme,
but we need you to read the requirements more carefully.
If the requirements are unclear, let us know so that we can continue to improve them.

When a ticket is closed as not approved, it looses it's position in the review queue.
When the update is submitted, a new ticket is created at the end of the queue.

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

https://make.wordpress.org/themes/handbook/review/how-to-do-a-review-draft/

https://github.com/WPTRT/theme-sniffer/

Use this upload form to submit your update: https://wordpress.org/themes/upload/


The theme looks broken...


  1. All Theme Sniffer reported Errors need to be fixed. Test against PHP 5.6 if you don't have backwards compatibility checks.
  1. Customizer
    • Header Image doesn't work.
  1. functions.php
    • L23, false
    • L295, escape
    • L342, plugin territory
  1. You're using the wrong textdomain
  1. lightwppizza-template-functions.php
    • L10, escape
    • L31, alt needs to be escaped and translatable
    • L35,59, escape, those are theme mods.
    • L97, escape
    • L94, that's not an attr
  1. lightwppizza-gutenberg.php
    • You are not allowed to register blocks, plugin territory. Please remove any such functionality.
  1. lightwppizza-functions.php
    • L20, preg_replace( '/[^\d+]/', '', $phone );
  1. lightwppizza-customizer.php
    • L34, remove the default
    • L181-238, esc_url_raw
  1. lightwppizza-woocommerce-template-functions.php
    • recheck escaping/sanitization
  1. header.php
    • L59, home_url( '/' )
    • L39, escape
  1. footer.php
    • L13,21,27,54,88,100,113 escape
    • L38, date_i18n( __( 'Y' , 'textdomain' ) )
  1. Theme URL is invalid - 404
  1. The license you mention on your website, CC by 3.0, is not GPL compatible.
  1. readme.txt
    • remove L4
    • Missing license information for any of the assets use in this theme

  1. Skip link - not ok
  1. Keyboard navigation - not ok

#3 in reply to: ↑ 2 @lightwp
6 years ago

Hi @acosmin

Thank you. I will try to fix these bugs...

Note: See TracTickets for help on using tickets.