[theme-reviewers] [WordPress Themes] #2478: THEME: Black Queen- 1.3

Philip M. Hofer (Frumph) philip at frumph.net
Mon Jan 10 19:09:53 UTC 2011


Hi Ford, if you could just reply to the ticket, it will send mails automatically to those involved with the ticket, there's no need to reply to the theme-reviewers list with your comments.

 - Phil

  ----- Original Message ----- 
  From: Ford Mon 
  To: theme-reviewers at lists.wordpress.org 
  Sent: Monday, January 10, 2011 10:50 AM
  Subject: Re: [theme-reviewers] [WordPress Themes] #2478: THEME: Black Queen- 1.3


  To  joshstauffer


  * style.css - #content .archive-meta has no unit specified. font-size:14; FIXED




  * Galleries with more than three columns aren't shown correctly. Test
   [gallery columns="4"] and it breaks the gallery. The
   bq_remove_gallery_css() removes inline styling and width of 33% is coded
   in style.css. The width should vary depending on the number of columns.  Again, this is not possible for my theme to allow 4 columns and it's not needed. 



  On Sun, Jan 9, 2011 at 8:35 PM, Ford Mon <fordmon at gmail.com> wrote:

    I am not able to adjust margins to allow 4 columns.. 3 columns is the best choice for my theme. For example this theme http://wordpress.org/extend/themes/fudo has been accepted several weeks ago and it has the same problem with 4 columns. 



    On Sun, Jan 9, 2011 at 4:52 PM, WordPress Themes <theme-reviewers at lists.wordpress.org> wrote:

      #2478: THEME: Black Queen - 1.3
      ------------------------------+-------------------------------
       Reporter:  themesfromrobin  |     Owner:  joshstauffer
           Type:  theme            |    Status:  assigned
      Resolution:                   |  Keywords:  theme-black-queen,
      ------------------------------+-------------------------------


      Comment (by joshstauffer):

       '''Requirements'''

       * Additional stylesheets should be appropriately loaded using
       wp_enqueue_style():
       http://codex.wordpress.org/Function_Reference/wp_enqueue_style

       * Galleries with more than three columns aren't shown correctly. Test
       [gallery columns="4"] and it breaks the gallery. The
       bq_remove_gallery_css() removes inline styling and width of 33% is coded
       in style.css. The width should vary depending on the number of columns.

       * style.css - #content .archive-meta has no unit specified. font-size:14;

       '''Recommendations'''

       * style.css - #content img has width: auto; and height: auto; which are
       commented but should be removed.

       * use get_stylesheet_uri() instead of bloginfo( 'stylesheet_url' )

      --
      Ticket URL: <http://themes.trac.wordpress.org/ticket/2478#comment:2>

      WordPress Themes <http://themes.trac.wordpress.org/>
      WordPress.org Theme Directory Reviews







------------------------------------------------------------------------------


  _______________________________________________
  theme-reviewers mailing list
  theme-reviewers at lists.wordpress.org
  http://lists.wordpress.org/mailman/listinfo/theme-reviewers
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110110/002f1141/attachment.htm>


More information about the theme-reviewers mailing list