-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4991/#review7681
-----------------------------------------------------------

Ship it!


The rave.less and rave-customizations.less files will need our standard Apache 
header since these are pure Rave files.  I will add when I apply the patch.

- Anthony


On 2012-05-08 13:08:28, Stan Drozdetski wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/4991/
> -----------------------------------------------------------
> 
> (Updated 2012-05-08 13:08:28)
> 
> 
> Review request for rave.
> 
> 
> Summary
> -------
> 
> This patch is the *first* step towards using LESS to compile Twitter 
> Bootstrap together with Rave stylesheets.
> 
> What it includes:
> - Inclusion of source Bootstrap files
> - LESS compilation of Bootstrap + Rave CSS (default.css and rave.css) done by 
> wro4j
> - Customization-friendly separation: Rave definitions are contained in 
> rave.less, while integrators can customize rave-customizations.less as they 
> see fit
> 
> What it DOES not include:
> - Factoring out "old" CSS files (although they are no longer referenced from 
> the <head>)
> - Reworking actual CSS definitions - much cleanup/consolidation remains
> - A solution to let developers compile LESS client-side (for easier 
> experimentation in debug mode)
> - Minification of CSS (would like to tackle that after we have a debug 
> option, which will turn off minification)
> 
> 
> Diffs
> -----
> 
>   https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/pom.xml 
> 1335493 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/resources/wro.properties
>  1335493 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/resources/wro.xml
>  1335493 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/tags/rave_css.tag
>  1335493 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/LICENSE
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/css/bootstrap-responsive.css
>  1335493 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/css/bootstrap.css
>  1335493 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/accordion.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/alerts.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/bootstrap.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/breadcrumbs.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/button-groups.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/buttons.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/carousel.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/close.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/code.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/component-animations.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/dropdowns.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/forms.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/grid.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/hero-unit.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/labels-badges.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/layouts.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/mixins.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/modals.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/navbar.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/navs.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/pager.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/pagination.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/popovers.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/progress-bars.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/reset.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/responsive-1200px-min.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/responsive-767px-max.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/responsive-768px-979px.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/responsive-navbar.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/responsive-utilities.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/responsive.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/scaffolding.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/sprites.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/tables.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/thumbnails.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/tooltip.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/type.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/utilities.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/variables.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/bootstrap/less/wells.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/rave-customizations.less
>  PRE-CREATION 
>   
> https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/css/rave.less
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/4991/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Stan
> 
>

Reply via email to