[theme-reviewers] Theme Uploader and CSS Classes

[Philip M. Hofer (Frumph)]

I look at it more like Otto just snagging the regexp stuff and ignoring the underlying reason to use it, the reporting back of information.

'steal bits' is pretty much just stripping out stuff to put into the wordpress one, otto can create his own code for that, the functionality of pross's is what would be good to use for the wordpress site.  If that's not going to be done, then just leave it on pross's.


  ----- Original Message ----- 
  From: Edward Caissie 
  To: theme-reviewers at lists.wordpress.org 
  Sent: Sunday, September 26, 2010 8:36 PM
  Subject: Re: [theme-reviewers] Theme Uploader and CSS Classes


  I'm sure there's a way to merge the best of both so it can be used in both places ...

  Pross' has more of the Theme Review items the Theme Review Team wants added to the uploader checks while the uploader (I imagine) has more script checking items than Pross'

  ... at least that's my view on it.


  Cais.


  On Sun, Sep 26, 2010 at 11:08 PM, Philip M. Hofer (Frumph) <philip at frumph.net> wrote:

    Steal bits? hrm.   Naw, let's just leave it up on Pross's website if thats the attitude your going to take on it.   Either use it or don't.





    ----- Original Message ----- From: "Otto" <otto at ottodestruct.com>

    To: <theme-reviewers at lists.wordpress.org>
    Sent: Sunday, September 26, 2010 2:31 PM

    Subject: Re: [theme-reviewers] Theme Uploader and CSS Classes



      If you feel like emailing me your code, I'll be happy to take a look
      at it and steal bits for the image uploader. Save us some time. ;)

      -Otto



      On Sun, Sep 26, 2010 at 11:52 AM, Simon Prosser <pross at pross.org.uk> wrote:

        On 09/25/2010 11:34 PM, Edward Caissie wrote:

          Yes, that would be very good!

          That way as items change their importance we can simply change the
          recommended to required or vice-versa.

        ok new version uploaded, now checks for php shortags, added recommended
        and required to functions. Also checks style.css for the alignment css

        --
        My Blog: http://www.pross.org.uk/
        Plugins : http://www.pross.org.uk/plugins/
        Themes: http://wordpress.org/extend/themes/profile/pross
        _______________________________________________
        theme-reviewers mailing list
        theme-reviewers at lists.wordpress.org
        http://lists.wordpress.org/mailman/listinfo/theme-reviewers


      _______________________________________________
      theme-reviewers mailing list
      theme-reviewers at lists.wordpress.org
      http://lists.wordpress.org/mailman/listinfo/theme-reviewers




    _______________________________________________
    theme-reviewers mailing list
    theme-reviewers at lists.wordpress.org
    http://lists.wordpress.org/mailman/listinfo/theme-reviewers





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


  _______________________________________________
  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/20100926/0b85d3cf/attachment-0001.htm>