<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Hi Atreyee,<br>
<br>
My overall comments:<br>
<br>
1) The "Host"/"Virtualization" tabs are not being displayed as a
Tab.<br>
You need to its background and also the first level second level
tab. It is the same on imaging the blue squares in the image below
filled with blue color. (does that make sense?)<br>
To distinguish a selected tab in the second level we can use a
darker color, ie, a darker blue (for ginger) or a darker green (for
kimchi).<br>
<br>
2) The first and the seconds level of tabs will have the same color.
For Kimchi, we continue to have one different color per tab which is
odd IMO.<br>
<br>
3) I'd suggest to align the "localhost" text with the first tab in
the second level. So the "Host" tab will be left shifted to have the
"Host" text aligned with "localhost".<br>
The same for Kimchi. You can check the red lines in the images
below.<br>
<br>
4) More comments in the patch related to the code.<br>
<br>
<img src="cid:part1.02050701.01000707@linux.vnet.ibm.com" alt=""><br>
<img src="cid:part2.05090008.03000104@linux.vnet.ibm.com" alt=""><br>
<br>
<div class="moz-cite-prefix">On 11/11/2015 20:34,
<a class="moz-txt-link-abbreviated" href="mailto:atreyee@linux.vnet.ibm.com">atreyee@linux.vnet.ibm.com</a> wrote:<br>
</div>
<blockquote
cite="mid:1447281287-2042-1-git-send-email-atreyee@linux.vnet.ibm.com"
type="cite">
<pre wrap="">From: Atreyee Mukhopadhyay <a class="moz-txt-link-rfc2396E" href="mailto:atreyee@linux.vnet.ibm.com"><atreyee@linux.vnet.ibm.com></a>
This patch is to implement new feature mentioned in issue #746.
First level navigation bar with Host and Virtualization tabs.
Second level tab navigation based on first level navigation.
Atreyee Mukhopadhyay (1):
Issue #746 : New navigation bar.
src/wok/plugins/gingerbase/ui/config/tab-ext.xml | 1 +
src/wok/plugins/kimchi/ui/config/tab-ext.xml | 1 +
ui/css/src/modules/_toolbar.scss | 9 ++-
ui/css/src/modules/_wok-variables.scss | 7 +-
ui/js/src/wok.main.js | 81 +++++++++++++++++++++---
ui/pages/wok-ui.html.tmpl | 6 +-
6 files changed, 91 insertions(+), 14 deletions(-)
</pre>
</blockquote>
<br>
</body>
</html>