Make WordPress Themes

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#6345 closed theme (not-approved)

THEME: lynx - 1.1.0

Reported by: john_wdl Owned by: SeizedPropaganda
Priority: Keywords: theme-lynx
Cc: john@…

Attachments (2)

lynx_header.png (112.7 KB) - added by SeizedPropaganda 6 years ago.
lynx_description.png (53.2 KB) - added by SeizedPropaganda 6 years ago.

Download all attachments as: .zip

Change History (16)

#1 @sabreuse
6 years ago

  • Owner set to sabreuse
  • Status changed from new to assigned

#2 @SeizedPropaganda
6 years ago


Are you going to be completing this review? I will wait a couple of days for a response before re-assigning the ticket to another reviewer.

#3 @SeizedPropaganda
6 years ago

  • Owner changed from sabreuse to SeizedPropaganda

#4 @SeizedPropaganda
6 years ago

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


  • Author URL is not acceptable. Per the Theme Review codex: Author URI, if used, is required to link to an author's personal web site or project/development website

More info: http://codex.wordpress.org/Theme_Review#Credit_Links

#5 @john_wdl
6 years ago


whats wrong with webdesignlessons.com ? all my other themes have that link and are approved.
Its just a webdesign/wp resource blog.


#6 @SeizedPropaganda
6 years ago

I'm going to consult with another review team member, I'll get back to you.

#7 @SeizedPropaganda
6 years ago

  • Resolution not-approved deleted
  • Status changed from closed to reopened

#8 @SeizedPropaganda
6 years ago

  • Status changed from reopened to assigned

#9 @SeizedPropaganda
6 years ago

Alright after consulting with another reviewer your author URL is fine, my apologies. I will complete the review and post my notes below.

#10 @john_wdl
6 years ago

Hello Tyler,

No need to apologize :)
Thanks for the review....

#11 @SeizedPropaganda
6 years ago

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

Theme Check - PASS:

RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
RECOMMENDED: No reference to add_custom_image_header was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.
RECOMMENDED: No reference to add_custom_background() was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.

Debugger results: PASSED

Validation: PASS

Deprecated Calls Plugin Results: PASSED

Theme Unit Test data results: FAILED

  • A big menu completely blows out the nav area. See my screenshot.
  • A long description blows out the header into the nav. See my screenshot

Code Quality: FAILED

  • Hook your lynx_add_javascript function into wp_enqueue_scripts:
add_action( 'wp_enqueue_scripts', 'lynx_add_javascript' );
  • Hook your lynx_the_widgets_init function into widgets_init:
add_action( 'widgets_init", 'lynx_the_widgets_init' );

Other Recommendations:

  • This is a complete review. If there are any issues once resolved, please re-submit your Theme using the uploader tool on wordpress.org/extend/themes, and a new Trac ticket will be generated.
  • Note: Re-submitted Themes are given review priority over new Themes.
  • You can use the following tools to recreate the issues raised above if any have been:
  • Themes should be reviewed using define('WP_DEBUG', true); in wp-config.php
  • You can also view the Theme Review page.
  • Theme developer questions, comments, and feedback are welcome and encouraged. Please use one of the following methods:
    • 1) Leave a comment on this ticket
    • 2) Email the theme-reviewers email list
    • 3) Use the #wordpress-themes channel on irc.freenode.ne

Recommended Action for this theme: NOT-APPROVED

#12 @john_wdl
6 years ago


In my earlier themes, i was asked to make a note in readme file saying the top menu only supports a few links, which i did and is present in this theme too.

Can i make a similar note saying descriptions also supports a few words?

The javascript and widgets_int are already hooked in includes/theme-init.php


#13 @SeizedPropaganda
6 years ago

My apologies I didn't notice that when I was reviewing the theme-init file (I typically put my add_action calls before or after the function so I usually look there).

Yes you can do that, I thought I mentioned that in the review but I missed that part. Adjust your readme and add those two things and I don't think there's anything else holding this theme back.

#14 @john_wdl
6 years ago


Here is the new ticket :


Note: See TracTickets for help on using tickets.