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

