From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Shirley Wang <swang(at)pivotal(dot)io> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [Design update] Style guide for pgAdmin4 |
Date: | 2017-04-24 09:00:58 |
Message-ID: | CA+OCxoyns0tAmnznnAq+Zr6Tp185Yu2rda1Yaw_q79F1VmQpzA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Fri, Apr 21, 2017 at 7:36 PM, Shirley Wang <swang(at)pivotal(dot)io> wrote:
> One more thing: there are two very close greens between notifiers and
> toggle buttons in preferences menus
>
> [image: notifications_switches.png]
>
> I recommend documenting the green for switches (#5cb85c), which would
> change notifiers from this:
>
> [image: Notifiers.png]
>
> to this:
> [image: Screen Shot 2017-04-20 at 5.28.55 PM.png]
>
Wouldn't it be better if it matched the (possibly-recoloured-by-us)
Bootstrap alert-success class? I agree that it's silly having two closely
matched colours, but I think it makes more sense to have the alerts and
notifiers match, than to match the switches.
> On Fri, Apr 21, 2017 at 2:27 PM Shirley Wang <swang(at)pivotal(dot)io> wrote:
>
>> Hello
>>
>>
>>>>>> Please feel free to edit the Sketch file- my only ask is new versions
>>>>>> hare named accordingly.
>>>>>>
>>>>>
>>>>> Is this the app you used? https://www.sketchapp.com/
>>>>>
>>>>
>>>> Yes. Also, I recognize it's probably not the best tool, given that
>>>> there's a 30 day free trial and then you need to purchase a license. I'm
>>>> looking into other programs that are free.
>>>>
>>>
>>> FYI, we use LucidCharts internally. It's a pretty good Google Apps
>>> integrated version of Microsoft Visio.
>>>
>>
>> Ah ok. Do you have any files related to pgAdmin you can share (or put
>> into Google drive folder)?
>>
>>
>>>
>>>
>>>>
>>>>
>>>>
>>>>> Some initial thoughts on the inventory:
>>>>>
>>>>> - Wow, that's a lot of colours. I didn't realise we had so many. I
>>>>> think we need to work that down to a set of a dozen or less primary colours.
>>>>>
>>>>
>>>> Me neither! I'm not so concerned with the number of different colors,
>>>> but with the number of similar shades of each color.
>>>>
>>>> I can distill those down and send another email shortly.
>>>>
>>>>
>>>>>
>>>>>
>>>> - I wouldn't worry about the button colour borders and gradients too
>>>>> much. Those are standard bootstrap colours, so we should document them in
>>>>> terms of bootstrap styles, not colour components.
>>>>>
>>>>
>>>> Ok. But are they appropriate for this app?
>>>>
>>>
>>> Personally I think they're fine.
>>>
>>
>> They seem ok for buttons (cancel, save, reset, toolbar). I found the
>> gradient is also on alerts when you click Dependencies, Dependents while
>> selecting Server in the browser.
>>
>> I think those alerts should not have a gradient to match with the other
>> types of alerts (notifiers and error messages). Since these specific alerts
>> are not success / fail messages, the blue works because it's a neutral
>> color.
>>
>> [image: highlight_selection color.png]
>> Here is what the updated colors would look like in the app.
>>
>> *Focus on text field in nodes*
>> [image: Screen Shot 2017-04-21 at 12.13.53 PM.png]
>>
>> *Browser*
>>
>> *[image: Screen Shot 2017-04-21 at 12.14.55 PM.png]*
>>
>> *Highlight in dropdown menu*
>>
>> *[image: Screen Shot 2017-04-21 at 12.19.51 PM.png]*
>>
>> *Alerts* [image: Screen Shot 2017-04-21 at 11.29.11 AM.png]
>> If this is ok, we will add these changes to our backlog as style changes.
>>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2017-04-24 10:57:26 | pgAdmin 4 commit: Added dependency on Flask-Migrate added by previous c |
Previous Message | Dave Page | 2017-04-24 08:58:11 | Re: [Design update] Style guide for pgAdmin4 |