[ovirt-devel] New design for the Gerrit UI

Sun May 7 07:34:31 UTC 2017

The current scheme seems a little buggy on FF. See how the top green bar is
not stretched to cover the top of the oVirt logo.

On 6 May 2017 at 12:49, Yaniv Kaul <ykaul at redhat.com> wrote:

>
>
> On Fri, May 5, 2017 at 5:33 PM, Greg Sheremeta <gshereme at redhat.com>
> wrote:
>
>> Here's my take.
>>
>> The line on the top is a little thicker than I like, but we have the same
>> thickness in oVirt (I checked). I'm ok with shrinking it in both places.
>>
>> I put in the logo from the oVirt login page. I added patternfly and
>> bootstrap. Changed the font to OpenSans. Positioned things and fixed
>> paddings. Changed the button and form field to patternfly type (blue button)
>>
>
> Looks very good now. I wonder how far are we from being ADA compliant
> (also a general question to our UI - I know Patternfly should help us there
> as well).
> Y.
>
>
>
>>
>> Best wishes,
>> Greg
>>
>>
>> On Fri, May 5, 2017 at 2:15 AM, Roy Golan <rgolan at redhat.com> wrote:
>>
>>>
>>> Adding UX people +Eldan Hildesheim <info at eldanet.com> +Alexander Wels
>>> <awels at redhat.com> +Greg Sheremeta <gshereme at redhat.com>
>>>
>>> On Fri, May 5, 2017 at 2:32 AM Martin Perina <mperina at redhat.com> wrote:
>>>
>>>> On Thu, May 4, 2017 at 1:13 PM, Nir Soffer <nsoffer at redhat.com> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Thu, May 4, 2017 at 4:26 PM Evgheni Dereveanchin <
>>>>> ederevea at redhat.com> wrote:
>>>>>
>>>>>> Thanks everyone for the great feedback!
>>>>>>
>>>>>> So there's two options I see now:
>>>>>> 1) keep the default header scheme with white background, just add the
>>>>>> project logo into the corner
>>>>>> 2) try to adapt to the Patternfly scheme as used in oVirt's Admin UI
>>>>>> currently.
>>>>>>
>>>>>> I've swapped the header background color to #393f45 as used in oVirt
>>>>>> for a quick test:
>>>>>> https://gerrit-staging.phx.ovirt.org/
>>>>>>
>>>>>
>>>>> Looks good!
>>>>>
>>>>
>>>> Much better for my eyes now, thanks a lot!
>>>> 
>>>>
>>>>
>>>>>
>>>>> The oVirt logo needs little more space around it, and it also should
>>>>> be centered vertically.
>>>>> Modifying the logo margin to 8px and width to 108px works for me using
>>>>> chrome,
>>>>> see attached screenshot.
>>>>>
>>>>> [image: Screenshot from 2017-05-04 20-11-39.png]
>>>>>
>>>>> Nir
>>>>>
>>>>> Is this more readable? If yes - I can continue working in this
>>>>>> direction to add gradients
>>>>>> and other patternfly style elements.
>>>>>>
>>>>> Otherwise I'll just go with option 1 and stick to the default style we
>>>>>> have now.
>>>>>>
>>>>>> On Thu, May 4, 2017 at 2:45 PM, Martin Sivak <msivak at redhat.com>
>>>>>> wrote:
>>>>>>
>>>>>>> > It will help if someone can suggest an alternate CSS which we can
>>>>>>> use or specific color codes,
>>>>>>>
>>>>>>> Well.. keep it as it is or make it really dark (like the patternfly
>>>>>>> menu). I do not care about logos but big area filled with non-neutral color
>>>>>>> is always going to be an issue.
>>>>>>>
>>>>>>> Martin
>>>>>>>
>>>>>>> On Thu, May 4, 2017 at 2:15 PM, Eyal Edri <eedri at redhat.com> wrote:
>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> On Thu, May 4, 2017 at 3:05 PM, Martin Perina <mperina at redhat.com>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>>> I agree with Milan and Martin, even after few minutes looking at
>>>>>>>>> it, the green
>>>>>>>>> with combination of white background just made my eyes burning :-(
>>>>>>>>>
>>>>>>>>> Would it be possible to use more darker colors (at least for top
>>>>>>>>> banner/menu)?
>>>>>>>>> For example darker colors we use in oVirt engine welcome page ...
>>>>>>>>>
>>>>>>>>
>>>>>>>> Thanks for the feedback,
>>>>>>>> It will help if someone can suggest an alternate CSS which we can
>>>>>>>> use or specific color codes,
>>>>>>>> otherwise it will be long trial and error process until we'll find
>>>>>>>> something that will suite everyone.
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Martin
>>>>>>>>>
>>>>>>>>> On Thu, May 4, 2017 at 5:53 AM, Martin Sivak <msivak at redhat.com>
>>>>>>>>> wrote:
>>>>>>>>>
>>>>>>>>>> I agree with Milan here. The light green background makes the menu
>>>>>>>>>> items to be almost unreadable, the search button (slightly
>>>>>>>>>> different
>>>>>>>>>> green color) blends with the background and generally the color
>>>>>>>>>> pulls
>>>>>>>>>> my eyes away from the content. I wouldn't feel comfortable
>>>>>>>>>> looking at
>>>>>>>>>> the screen for a whole day.
>>>>>>>>>>
>>>>>>>>>> Martin
>>>>>>>>>>
>>>>>>>>>> On Thu, May 4, 2017 at 9:57 AM, Milan Zamazal <
>>>>>>>>>> mzamazal at redhat.com> wrote:
>>>>>>>>>> > Evgheni Dereveanchin <ederevea at redhat.com> writes:
>>>>>>>>>> >
>>>>>>>>>> >> The Infra team is working on customizing the look of Gerrit to
>>>>>>>>>> make it fit
>>>>>>>>>> >> better with other oVirt services. I want to share the result
>>>>>>>>>> of this
>>>>>>>>>> >> effort. Hopefully we can gather some feedback before applying
>>>>>>>>>> the design to
>>>>>>>>>> >> oVirt's instance of Gerrit.
>>>>>>>>>> >>
>>>>>>>>>> >> Please visit the Staging instance to check it out:
>>>>>>>>>> >>
>>>>>>>>>> >>   https://gerrit-staging.phx.ovirt.org/
>>>>>>>>>> >
>>>>>>>>>> > Thank you for the preview.  While it fits better with oVirt
>>>>>>>>>> services,
>>>>>>>>>> > there is one thing that makes me uncomfortable with it: low
>>>>>>>>>> contrast.
>>>>>>>>>> > The top green bar is probably directly violating Web
>>>>>>>>>> Accessibility
>>>>>>>>>> > Guidelines (AA level; see
>>>>>>>>>> > https://www.w3.org/TR/WCAG20/#visual-audio-contrast-contrast),
>>>>>>>>>> but I
>>>>>>>>>> > find all the green parts harder to read than in the current
>>>>>>>>>> version.
>>>>>>>>>> > So it would be nice if the contrast could be improved.
>>>>>>>>>> >
>>>>>>>>>> > Thanks,
>>>>>>>>>> > Milan
>>>>>>>>>> > _______________________________________________
>>>>>>>>>> > Devel mailing list
>>>>>>>>>> > Devel at ovirt.org
>>>>>>>>>> > http://lists.ovirt.org/mailman/listinfo/devel
>>>>>>>>>> _______________________________________________
>>>>>>>>>> Devel mailing list
>>>>>>>>>> Devel at ovirt.org
>>>>>>>>>> http://lists.ovirt.org/mailman/listinfo/devel
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> _______________________________________________
>>>>>>>>> Infra mailing list
>>>>>>>>> Infra at ovirt.org
>>>>>>>>> http://lists.ovirt.org/mailman/listinfo/infra
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>>
>>>>>>>> Eyal edri
>>>>>>>>
>>>>>>>>
>>>>>>>> ASSOCIATE MANAGER
>>>>>>>>
>>>>>>>> RHV DevOps
>>>>>>>>
>>>>>>>> EMEA VIRTUALIZATION R&D
>>>>>>>>
>>>>>>>>
>>>>>>>> Red Hat EMEA <https://www.redhat.com/>
>>>>>>>> <https://red.ht/sig> TRIED. TESTED. TRUSTED.
>>>>>>>> <https://redhat.com/trusted>
>>>>>>>> phone: +972-9-7692018 <+972%209-769-2018>
>>>>>>>> irc: eedri (on #tlv #rhev-dev #rhev-integ)
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Devel mailing list
>>>>>>> Devel at ovirt.org
>>>>>>> http://lists.ovirt.org/mailman/listinfo/devel
>>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Regards,
>>>>>> Evgheni Dereveanchin
>>>>>> _______________________________________________
>>>>>> Devel mailing list
>>>>>> Devel at ovirt.org
>>>>>> http://lists.ovirt.org/mailman/listinfo/devel
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Devel mailing list
>>>>> Devel at ovirt.org
>>>>> http://lists.ovirt.org/mailman/listinfo/devel
>>>>>
>>>> _______________________________________________
>>>> Infra mailing list
>>>> Infra at ovirt.org
>>>> http://lists.ovirt.org/mailman/listinfo/infra
>>>>
>>>
>>
>>
>> --
>> Greg Sheremeta, MBA
>> Sr. Software Engineer
>> Red Hat, Inc.
>> gshereme at redhat.com
>>
>> _______________________________________________
>> Devel mailing list
>> Devel at ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/devel
>>
>
>
> _______________________________________________
> Infra mailing list
> Infra at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
>
>

-- 
Barak Korren
RHV DevOps team , RHCE, RHCi
Red Hat EMEA
redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20170507/bcdb3f6d/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screenshot from 2017-05-04 20-11-39.png
Type: image/png
Size: 83036 bytes
Desc: not available
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20170507/bcdb3f6d/attachment-0002.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screenshot from 2017-05-07 10-31-49.png
Type: image/png
Size: 63894 bytes
Desc: not available
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20170507/bcdb3f6d/attachment-0003.png>