tmortagne (SVN) wrote:
> Author: tmortagne
> Date: 2008-12-15 18:26:24 +0100 (Mon, 15 Dec 2008)
> New Revision: 14755
> 
> Modified:
>    platform/skins/trunk/albatross/src/main/resources/albatross/wiki.css
> Log:
> XSALBATROSS-44: Add support for BoxMacro
> 
> Modified: platform/skins/trunk/albatross/src/main/resources/albatross/wiki.css
> ===================================================================
> --- platform/skins/trunk/albatross/src/main/resources/albatross/wiki.css      
> 2008-12-15 16:45:22 UTC (rev 14754)
> +++ platform/skins/trunk/albatross/src/main/resources/albatross/wiki.css      
> 2008-12-15 17:26:24 UTC (rev 14755)
> @@ -75,6 +75,20 @@
>    font-size: inherit;
>  }
>  
> +.box {
> +  margin-top: 4px;
> +  margin-bottom: 4px;

Could be replaced with the shorter:
margin: 0 4px;

> +  padding: 5px 5px 5px 5px;

Could be replaced with the shorter:
padding: 5px;

> +  color: inherit;

That's useless, since this is the default. Should be removed. If there's 
another CSS rule that overrides this, than that rule should be either 
removed or narrowed down to a more specific set of elements.

> +  background-color: #eeeeee;

Could be replaced with the shorter:
background-color: #EEE;

> +  border: 1px dotted #003366;

Could be replaced with the shorter:

border: 1px dotted #036;

> +  font-size: 12px;

I prefer not to use px for font sizes, since this prevents increasing 
the font size in IE6, for those that don't see so well.

> +  line-height: 100%;
> +  white-space: pre;
> +  width: 70%;
> +  overflow: auto;
> +}

-- 
Sergiu Dumitriu
http://purl.org/net/sergiu/
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to