[Ovirt-devel] [PATCH server] UI updates/tweaks. [rebased]

[Jason Guiditta]

ACK, with one requested change inline.  If you dont have push perms,
resend with fix and I will push for you.

-j

On Wed, 2009-05-06 at 12:53 -0400, Jeremy Perry wrote:
> From: Jeremy Perry <jperry at redhat.com>
> 
> New images integrated, almost all images in the UI are updated or replaced. Also includes several  minor style tweaks and in a few cases some verbage and capitalization changes for consistency.
> 
<snip>
>  
> 
> @@ -199,8 +206,9 @@ a { color:#000000; text-decoration: none;}
>  	background: none;
>  
>  }
> -.ui-tabs-nav a:hover {color:#FFFFFF; border-bottom: 5px solid #5599B8;}
> -.ui-tabs-selected a {color:#FFFFFF; font-weight: bold; height: 24px; line-height:1.6; border-bottom: 5px solid #5599B8;}
> +.ui-tabs-nav a:hover {color:#FFFFFF; border-bottom: 5px solid #537585;}
> +.ui-tabs-nav a:focus, .ui-tabs-nav a:active {outline-style: none;}
> +.ui-tabs-selected a {color:#FFFFFF; outline-style: none; height: 24px; line-height:1.6; border-bottom: 5px solid #5599B8;}
>  
>  .ui-tabs-hide {/*display:none;*/}
>  
> @@ -276,8 +284,8 @@ a { color:#000000; text-decoration: none;}
>    float:left;
>    line-height:1.7;
>    list-style-type:none;
> -  margin:0pt;
> -  padding:0pt 8px;
> +  margin:0;
> +  padding:0;
>    text-align:center;
>    text-decoration:none;
>  }
> @@ -298,6 +306,10 @@ a { color:#000000; text-decoration: none;}
>    text-decoration:none;
>  }
>  .selection_tools a:hover {
> +  -moz-border-radius-bottomleft:2px;
> +  -moz-border-radius-bottomright:2px;
> +  -moz-border-radius-topleft:2px;
> +  -moz-border-radius-topright:2px;

Would you be kind enough to combine the above into one line as follows?
-moz-border-radius: 2px;
>    background:#B0CBDE none repeat scroll 0%;
>  }
>  .selection_key {
> diff --git a/src/public/stylesheets/ovirt-tree/tree.css b/src/public/stylesheets/ovirt-tree/tree.css
> index 441ae83..ef1f1f7 100644
> --- a/src/public/stylesheets/ovirt-tree/tree.css
> +++ b/src/public/stylesheets/ovirt-tree/tree.css
> @@ -11,7 +11,7 @@
>  .ovirt-tree, .ovirt-tree ul {
>      list-style: none;
>      margin:0;
> -    padding:5px 0 5px 16px;
> +    padding:1px 0 1px 22px;
>      margin-right: 8px;
>  }
>  
> @@ -19,6 +19,7 @@
>      background-repeat: no-repeat;
>      background-position: left;
>      padding: 4px 0 4px 4px;
> +    width: 100%;
>      cursor: pointer; /* maybe move this into a clickable class? */
>  }
>  
> @@ -59,16 +60,16 @@
>  }
>  
>  .expandable {
> -    background: url('../../images/plus.gif') no-repeat left;
> +    background: url('../../images/dir_closed.png') no-repeat left;
>  }
>  
>  .expanded {
> -    background: url('../../images/minus.gif') no-repeat left;
> +    background: url('../../images/dir_open.png') no-repeat left;
>  }
>  
>  .ovirt-tree .current {
>      background-color: #698FA6;
> -    color:#000000;
> +    color:#FFFFFF;
>      width: 100%;
>  }
>