[theme-reviewers] Theme author is asking end user to install less php

Thu Jul 4 23:12:32 UTC 2013

Perhaps a more ideal solution would be to submit a core ticket/patch to get
a preprocessor in core? That way, core, Plugins, and Themes can rely on the
core-bundled preprocessor.

Until then, as Emil said: as long as the Theme works properly out of the
box, it's fine for Themes to ship with options/integration to support a
Plugin that adds a preprocessor.

On Thu, Jul 4, 2013 at 5:21 PM, Emil Uzelac <emil at uzelac.me> wrote:

> *I meant*
>
> No need to rely or require WP-LESS*
>
>
> On Thu, Jul 4, 2013 at 4:17 PM, Emil Uzelac <emil at uzelac.me> wrote:
>
>> If he ships the Theme with LESS as an additional (optional) feature and
>> the Theme
>> works out of the box as you and Chip mentioned, it's still good to go and
>> acceptable :)
>>
>> No need to rely or require on WP-LESS.
>>
>>
>> On Thu, Jul 4, 2013 at 4:01 PM, Josh Pollock <jpollock412 at gmail.com>wrote:
>>
>>> @Divesh-
>>>
>>> If you ship your theme with processed and unprocessed code, as well as
>>> directions in your readme for using wpless with your theme then all of
>>> the requirements (theme review guidelines and yours) will be met. That way
>>> it works out of the box, but the end user knows you used wpless and how to
>>> use it to modify your theme.
>>>
>>>
>>> On Thu, Jul 4, 2013 at 4:06 PM, Emil Uzelac <emil at uzelac.me> wrote:
>>>
>>>> even though if there's an option and the Theme works
>>>> right without the preprocessors is not the reason to fail
>>>> a Theme
>>>>
>>>>
>>>> On Thu, Jul 4, 2013 at 3:02 PM, Amy Hendrix <sabreuse at gmail.com> wrote:
>>>>
>>>>> I'd say that both Sass and LESS files should be accepted (because if
>>>>> you're a Sass or LESS user, it's far preferable to make any downstream
>>>>> changes to the original files) -- but the theme should be packaged
>>>>> with compiled css, and not depend on running any preprocessing live,
>>>>> whether it's directly on the server or by way of a WP plugin.
>>>>>
>>>>> Emil, the tutorial you point to is about using preprocessors in your
>>>>> development environment and compiling out the finished css. The
>>>>> original question was about requiring libraries to compile it live on
>>>>> the server, which should not be approved IMO (and which is a terrible
>>>>> idea for preprocessing performance, but that's a debate that's way out
>>>>> of scope for this list!)
>>>>>
>>>>> Personally, I love preprocessors; I think it's the way of the future
>>>>> and I maintain a library for working with Sass and _s. But asking
>>>>> users to install extra libraries just to be able to use a theme at all
>>>>> is _doing_it_wrong().
>>>>>
>>>>> On Thu, Jul 4, 2013 at 3:51 PM, Emil Uzelac <emil at uzelac.me> wrote:
>>>>> > Both Sass and LESS preprocessor should be accepted.
>>>>> > Not really sure if you need any extra PHP's for this.
>>>>> > Take a look at this brief tut.
>>>>> >
>>>>> >
>>>>> > On Thu, Jul 4, 2013 at 1:30 PM, Jose Castaneda <
>>>>> jomcastaneda at gmail.com>
>>>>> > wrote:
>>>>> >>
>>>>> >> The thing to remember about the WordPress theme repo is that not
>>>>> everybody
>>>>> >> that will use your theme is a developer. More often than not it is
>>>>> a person
>>>>> >> with little to no knowledge of web development or programming
>>>>> prowess. What
>>>>> >> I would do is include both the LESS files and the compiled CSS and
>>>>> leave a
>>>>> >> note on the readme file or put up some sort of notice.
>>>>> >>
>>>>> >>
>>>>> >> On Thu, Jul 4, 2013 at 7:54 AM, divesh kumar <
>>>>> diveshkumar1983 at gmail.com>
>>>>> >> wrote:
>>>>> >>>
>>>>> >>> hi greg
>>>>> >>>
>>>>> >>> but if i use compiled css it will be just a plain css and nobody
>>>>> would
>>>>> >>> get to know that its using lessphp.
>>>>> >>>
>>>>> >>> On Jul 4, 2013 5:15 PM, "Srikanth Koneru" <tskk79 at gmail.com>
>>>>> wrote:
>>>>> >>>>
>>>>> >>>> The basic theme must function without depending on any plugin.
>>>>> You can
>>>>> >>>> have additional features which can come live if plugins(that are
>>>>> required
>>>>> >>>> for those features) are activated.
>>>>> >>>> If lessphp framework is allowed in themes, you can bundle it with
>>>>> your
>>>>> >>>> theme itself.
>>>>> >>>>
>>>>> >>>>
>>>>> >>>> On Thu, Jul 4, 2013 at 4:52 PM, divesh kumar <
>>>>> diveshkumar1983 at gmail.com>
>>>>> >>>> wrote:
>>>>> >>>>>
>>>>> >>>>> Hi Kavita,
>>>>> >>>>>
>>>>> >>>>> Thanks for the reference. so that means user must install that
>>>>> plugin
>>>>> >>>>> in order to use my theme. Can i create any kind of dependency of
>>>>> my theme to
>>>>> >>>>> use this?
>>>>> >>>>>
>>>>> >>>>>
>>>>> >>>>>
>>>>> >>>>>
>>>>> >>>>> On Thu, Jul 4, 2013 at 4:26 PM, Kavita Kumawat <
>>>>> erkavecb at gmail.com>
>>>>> >>>>> wrote:
>>>>> >>>>>>
>>>>> >>>>>> you may want to check this.
>>>>> >>>>>>
>>>>> >>>>>> http://wordpress.org/plugins/wp-less/
>>>>> >>>>>>
>>>>> >>>>>> -Kavita
>>>>> >>>>>>
>>>>> >>>>>>
>>>>> >>>>>> On Thu, Jul 4, 2013 at 3:34 PM, divesh kumar
>>>>> >>>>>> <diveshkumar1983 at gmail.com> wrote:
>>>>> >>>>>>>
>>>>> >>>>>>> Guys,
>>>>> >>>>>>>
>>>>> >>>>>>> Does that means I cannot use SASS or any other CSS framework
>>>>> with
>>>>> >>>>>>> wordpress theme. I don't think this make sense.
>>>>> >>>>>>>
>>>>> >>>>>>> Thanks
>>>>> >>>>>>>
>>>>> >>>>>>>
>>>>> >>>>>>> On Thu, Jul 4, 2013 at 2:22 PM, Srikanth Koneru <
>>>>> tskk79 at gmail.com>
>>>>> >>>>>>> wrote:
>>>>> >>>>>>>>
>>>>> >>>>>>>> Themes should work out of the box.
>>>>> >>>>>>>>
>>>>> >>>>>>>>
>>>>> >>>>>>>> On Thu, Jul 4, 2013 at 2:19 PM, carolina poena
>>>>> >>>>>>>> <myazalea at hotmail.com> wrote:
>>>>> >>>>>>>>>
>>>>> >>>>>>>>> ticket: http://themes.trac.wordpress.org/ticket/12893
>>>>> >>>>>>>>> In readme:
>>>>> >>>>>>>>>
>>>>> >>>>>>>>> 1. Upload the `snAdaptive` folder to the
>>>>> `/wp-content/themes/`
>>>>> >>>>>>>>> directory
>>>>> >>>>>>>>> 2. Download lessphp framework from
>>>>> >>>>>>>>> http://leafo.net/lessphp/src/lessphp-0.3.9.tar.gz.
>>>>> >>>>>>>>> 3. Extract it in the directory themes/snAdaptive/lessphp.
>>>>> >>>>>>>>> 4. Make sure there is a file less.inc.php in directory
>>>>> >>>>>>>>> /themes/snAdaptive/lessphp/.
>>>>> >>>>>>>>> 5. Now you are ready to use the theme.
>>>>> >>>>>>>>>
>>>>> >>>>>>>>> The theme will not be approved eitherway but this is not ok
>>>>> right?
>>>>> >>>>>>>>> ^^
>>>>> >>>>>>>>>
>>>>> >>>>>>>>> //Carolina
>>>>> >>>>>>>>>
>>>>> >>>>>>>>>
>>>>> >>>>>>>>> _______________________________________________
>>>>> >>>>>>>>> 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
>>>>> >>>>>>
>>>>> >>>>>
>>>>> >>>>>
>>>>> >>>>> _______________________________________________
>>>>> >>>>> 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
>>>>> >>
>>>>> >
>>>>> >
>>>>> > _______________________________________________
>>>>> > 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
>>>
>>>
>>
>
> _______________________________________________
> 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/20130704/0fb72986/attachment-0001.html>