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