On Fri, Apr 21, 2017 at 7:36 PM, Shirley Wang <swang@pivotal.io> wrote:
One more thing: there are two very close greens between notifiers and toggle buttons in preferences menus
I recommend documenting the green for switches (#5cb85c), which would change notifiers from this:
to this:
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@pivotal.io> wrote:
Hello
Please feel free to edit the Sketch file- my only ask is new versions hare named accordingly.
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.
Here is what the updated colors would look like in the app.
Focus on text field in nodes
Browser
Highlight in dropdown menu
Alerts
If this is ok, we will add these changes to our backlog as style changes.