[Kimchi-devel] [PATCH] New Navigation bar

[Walter Niklaus]

Aline,  the snapshots you have taken and the problems you described make 
me suspect that the  scss changes didn't really make it into the css files.
Atreyee struggled last week with this issue till Samuel pointed out that 
she may have to delete the css files before issuing the make.
Chandra adressed this problem in Issue #757 and I think that patch is 
available in the meantime.

On 16.11.2015 15:28, Aline Manera wrote:
> Hi Atreyee,
>
> My overall comments:
>
> 1) The "Host"/"Virtualization" tabs are not being displayed as a Tab.
>     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?)
>     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).
>
> 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.
>
> 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".
>     The same for Kimchi. You can check the red lines in the images below.
>
> 4) More comments in the patch related to the code.
>
>
>
>
> On 11/11/2015 20:34, atreyee at linux.vnet.ibm.com wrote:
>> From: Atreyee Mukhopadhyay<atreyee at linux.vnet.ibm.com>
>>
>> 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(-)
>>
>
>
>
> _______________________________________________
> Kimchi-devel mailing list
> Kimchi-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/kimchi-devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/kimchi-devel/attachments/20151116/6e5ea723/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/jpeg
Size: 149349 bytes
Desc: not available
URL: <http://lists.ovirt.org/pipermail/kimchi-devel/attachments/20151116/6e5ea723/attachment.jpe>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/jpeg
Size: 98452 bytes
Desc: not available
URL: <http://lists.ovirt.org/pipermail/kimchi-devel/attachments/20151116/6e5ea723/attachment-0001.jpe>