Hi Simone,

 

recently I've been working on website performance improvements and you just hit 
another issue:

"Don't use import"[1]

We wanted to make only one call for css, so I wrote a small groovy-script which 
bundled all imports. 

I still have to make write a maven-plugin for it.

So it's up to you: use links or bundle them. This is really a response-killer, 
more than I could imagine!

 

-Robert

 

[1] http://www.stevesouders.com/blog/2009/04/09/dont-use-import/



http://www.stevesouders.com/blog/2009/04/09/dont-use-import/----------------------------------------
> Subject: svn commit: r1201702 - 
> /maven/sandbox/trunk/skin/maven-fluido-skin/src/main/resources/META-INF/maven/site.vm
> Date: Mon, 14 Nov 2011 13:47:07 +0000
> To: [email protected]
> From: [email protected]
>
> Author: simonetripodi
> Date: Mon Nov 14 13:47:06 2011
> New Revision: 1201702
>
> URL: http://svn.apache.org/viewvc?rev=1201702&view=rev
> Log:
> restored the default css importing policy
> restored the missing site.css for extension points
>
> Modified:
> maven/sandbox/trunk/skin/maven-fluido-skin/src/main/resources/META-INF/maven/site.vm
>
> Modified: 
> maven/sandbox/trunk/skin/maven-fluido-skin/src/main/resources/META-INF/maven/site.vm
> URL: 
> http://svn.apache.org/viewvc/maven/sandbox/trunk/skin/maven-fluido-skin/src/main/resources/META-INF/maven/site.vm?rev=1201702&r1=1201701&r2=1201702&view=diff
> ==============================================================================
> --- 
> maven/sandbox/trunk/skin/maven-fluido-skin/src/main/resources/META-INF/maven/site.vm
>  (original)
> +++ 
> maven/sandbox/trunk/skin/maven-fluido-skin/src/main/resources/META-INF/maven/site.vm
>  Mon Nov 14 13:47:06 2011
> @@ -507,7 +507,12 @@
> <head>
> <meta http-equiv="Content-Type" content="text/html; 
> charset=${outputEncoding}" />
> <title>$title</title>
> - <link rel="stylesheet" href="$relativePath/css/bootstrap-1.3.0.min.css" />
> + <style type="text/css" media="all">
> + @import url("$relativePath/css/bootstrap-1.3.0.min.css");
> + @import url("$relativePath/css/maven-base.css");
> + @import url("$relativePath/css/maven-theme.css");
> + @import url("$relativePath/css/site.css");
> + </style>
> #if ( $decoration.custom.getChild( 'fluidoSkin' )
> && $decoration.custom.getChild( 'fluidoSkin' ).getChild( 'topBarEnabled' )
> && 'true' == $decoration.custom.getChild( 'fluidoSkin' ).getChild( 
> 'topBarEnabled' ).getValue() )
> @@ -516,8 +521,6 @@
> #else
> <style>body{padding-top: 20px;}</style>
> #end
> - <link rel="stylesheet" href="$relativePath/css/maven-base.css" />
> - <link rel="stylesheet" href="$relativePath/css/maven-theme.css" />
>
> #if ( $decoration.body.head )
> #foreach( $item in $decoration.body.head.getChildren() )
>
>                                         
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to