On Mon, Dec 15, 2008 at 7:00 PM, Sergiu Dumitriu <[email protected]> wrote:
> 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;
>> +}

I just copy/pasted the .code class css code...

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



-- 
Thomas Mortagne
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to