[Yanel-dev] Re: [Yanel-commits] rev 41609 - public/yanel/trunk/src/webapp/htdocs

[Guillaume Déflache]

Hi!

michi at wyona.com schrieb:
> Author: michi
> Date: 2009-02-16 12:50:21 +0100 (Mon, 16 Feb 2009)
> New Revision: 41609
> 
> Modified:
>    public/yanel/trunk/src/webapp/htdocs/toolbar.css
> Log:
> haschild added and font set for various css items
> 
> Modified: public/yanel/trunk/src/webapp/htdocs/toolbar.css
> ===================================================================
> --- public/yanel/trunk/src/webapp/htdocs/toolbar.css	2009-02-16 11:20:42 UTC (rev 41608)
> +++ public/yanel/trunk/src/webapp/htdocs/toolbar.css	2009-02-16 11:50:21 UTC (rev 41609)
> @@ -43,6 +43,9 @@
>    margin-left:6px;
>    text-align:left;
>    font-size: 12px;
> +  font-weight: normal;
> +  font-style: normal;
> +  font-family: "lucida grande", tahoma, verdana, arial, sans-serif;
>  }

Should not that be
 > +  font-family: "Lucida Grande", Tahoma, Verdana, Arial, sans-serif;
? I am afraid some browsers may be case-sensitive...


>  #yaneltoolbar_menu a, #yaneltoolbar_menu #yaneltoolbar_menutitle{
> @@ -54,10 +57,18 @@
>  
>  #yaneltoolbar_menutitle {
>    padding-right:30px;
> +  font-size: 12px;
> +  font-weight: normal;
> +  font-style: normal;
> +  font-family: "lucida grande", tahoma, verdana, arial, sans-serif;
>  }
>  
>  #yaneltoolbar_menu a {
>    text-decoration:none;
> +  font-size: 12px;
> +  font-weight: normal;
> +  font-style: normal;
> +  font-family: "lucida grande", tahoma, verdana, arial, sans-serif;
>  }
>  
>  #yaneltoolbar_menu a, #yaneltoolbar_menu a:visited {
> @@ -91,13 +102,22 @@
>    white-space: nowrap;
>  }
>  
> -#yaneltoolbar_menu li{
> +#yaneltoolbar_menu li {
>    position:relative;
>    margin:1px;
>    padding:2px;
> +  font-size: 12px;
> +  font-weight: normal;
> +  font-style: normal;
> +  font-family: "lucida grande", tahoma, verdana, arial, sans-serif;
> +  color: grey;
>  }

Couldn't a class/id ("yaneltoolbar" comes to mind!) have been added to 
factor out all these duplicated font settings?


-- 
Cheers,
    Guillaume