Make WordPress Themes

Change History (6)

#1 @chipbennett
5 years ago

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

#2 @chipbennett
5 years ago

Previous-ticket required issues

  • Menu support/design for multiple-level menu items: fixed (added depth => 1)


  • Screenshot is not appropriate. (Recommend using the one on Theme URI)
  • Remove license link and CC text from footer


  • Pass sidebar ID rather than name/description, to dynamic_sidebar()/is_active_sidebar(), etc.

Review Summary

  • Theme has required issues, but they are minor only
  • I'll hold this ticket open to allow time to fix and resubmit
  • Please post a comment to this ticket, with a link to the new ticket, after you resubmit, and I will continue the review in the new ticket.

#3 @obert
5 years ago

Hi, i re-uploaded the 2.2 version of my theme: http://themes.trac.wordpress.org/ticket/11512 I hope that had fixed all issues correctly.
thanks for your time.

#4 @chipbennett
5 years ago

  • Resolution set to closed-newer-version-uploaded
  • Status changed from assigned to closed

#5 @obert
5 years ago

I forgot to replace sidebar name with ID in every is_active_sidebar. sorry: i noticed it only now.

#6 @obert
5 years ago

I uploaded the 2.3 version of this theme if you want to check it: http://themes.trac.wordpress.org/ticket/11618
Thanks again to had approved it.

Note: See TracTickets for help on using tickets.