Mailing List Archive: Bricolage: commits

[8559] Fixed a small CSS syntax error.

Index | Next | Previous | View Threaded

brewt at bricolage

Apr 6, 2009, 9:59 PM

Post #1 of 2 (737 views)
Permalink

[8559] Fixed a small CSS syntax error.

Revision: 8559
Author: brewt
Date: 2009-04-06 21:59:03 -0700 (Mon, 06 Apr 2009)
ViewCVS: http://viewsvn.bricolage.cc/?rev=8559&view=rev

Log Message:
-----------
Fixed a small CSS syntax error. I assume it shouldn't have a 1em top/bottom padding because it looked pretty bad like that.

Modified Paths:
--------------
bricolage/trunk/comp/media/css/style.css

Modified: bricolage/trunk/comp/media/css/style.css
===================================================================
--- bricolage/trunk/comp/media/css/style.css 2009-04-07 01:28:44 UTC (rev 8558)
+++ bricolage/trunk/comp/media/css/style.css 2009-04-07 04:59:03 UTC (rev 8559)
@@ -421,7 +421,7 @@
div.popup-menu h4 {
display: block;
font-size: .9em;
- padding: 1em, .2em;
+ padding: .2em;
margin: 0;
background: #EAEAEA;
color: #999;

adeola at digitalcraftsmen

Apr 6, 2009, 10:44 PM

Post #2 of 2 (679 views)
Permalink

Re: [8559] Fixed a small CSS syntax error. [In reply to]

On 7/4/09 05:59, brewt [at] bricolage wrote:
> --- bricolage/trunk/comp/media/css/style.css 2009-04-07 01:28:44 UTC (rev 8558)
> +++ bricolage/trunk/comp/media/css/style.css 2009-04-07 04:59:03 UTC (rev 8559)
> @@ -421,7 +421,7 @@
> div.popup-menu h4 {
> display: block;
> font-size: .9em;
> - padding: 1em, .2em;
> + padding: .2em;
> margin: 0;
> background: #EAEAEA;
> color: #999;
>
The fix for this should probably have been:

padding: 1em .2em;

Only removing the coma. Removing the `1em` rule would collapse the `top`
and `bottom` padding and might not look as intended. Unless the default
was equivalent to the removed dimension. Did you check that this
displayed correctly?

Regards,
Adeola

--
Creative Developer - Digital Craftsmen Ltd
Exmouth House, 3 Pine Street
London, EC1R 0JH
t: +44 20 7183 1410
f: +44 20 7099 5140
m: +44 75 9527 7886
w: http://www.digitalcraftsmen.net/

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads