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

Josh Pollock jpollock412 at gmail.com
Fri Jul 5 00:31:10 UTC 2013


@Chip

Do you think there is any chance of a LESS or SASS preprocesser getting
accepted into core if it was submitted?


On Thu, Jul 4, 2013 at 7:12 PM, Chip Bennett <chip at chipbennett.net> wrote:

> 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
>>
>>
>
> _______________________________________________
> 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/aa6b2b40/attachment-0001.html>


More information about the theme-reviewers mailing list