Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Tweak default grey to be darker, possibly colder #35

Closed
emilyhobson opened this issue Jan 9, 2017 · 1 comment
Closed

Tweak default grey to be darker, possibly colder #35

emilyhobson opened this issue Jan 9, 2017 · 1 comment
Assignees

Comments

@emilyhobson
Copy link

emilyhobson commented Jan 9, 2017

panel on the left and text headers on the table
Consider making higher constrast globally and possibly be more harmonious with grey used on map

@emilyhobson emilyhobson changed the title Tweak default grey to be darker Tweak default grey to be darker, possibly colder Jan 9, 2017
@almccon
Copy link
Member

almccon commented Jan 18, 2017

Currently the panel on the left is defined here: https://github.com/cityenergyproject/seattle/blob/master/src/styles/main/_filter.scss#L134
@include linear-gradient(to right, #dedede, #d0d0d0);

The text seems to be #d1d1d0 from here: https://github.com/cityenergyproject/seattle/blob/master/src/styles/base/_variables.scss#L21

If we try to match the map, here are a few colors to try:
#a4afb0 is the dark gray of the coastline in the basemap.
#cdd2d4 is the blue-gray of the ocean on the map

Option 1 is #a4afb0
screen shot 2017-01-17 at 9 22 02 pm

Option 2 is #cdd2d4
screen shot 2017-01-17 at 9 33 01 pm

Option 3 is just darkening the existing gray, which I think works better:
darkening the existing gray

Option 4 is the same as Option 3, but I'm darkening the pop-up, too. I think that makes the popup harder to see, however.
darkening the existing gray and popup

Going to implement Option 3 for now.

almccon added a commit that referenced this issue Jan 18, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants