WordPress.org

Make WordPress Themes

Change History (6)

comment:1 chipbennett14 months ago

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

comment:2 chipbennett14 months ago

Previous-ticket required issues

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

Required

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

Recommended

  • 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.

comment:3 obert14 months 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.
Alberto

comment:4 chipbennett14 months ago

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

comment:5 obert14 months ago

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

comment:6 obert14 months 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.