Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

UI feedback: Information hierarchy #51

Closed
muesliq opened this issue Jan 4, 2017 · 4 comments
Closed

UI feedback: Information hierarchy #51

muesliq opened this issue Jan 4, 2017 · 4 comments

Comments

@muesliq
Copy link
Collaborator

muesliq commented Jan 4, 2017

I like the UI, yet I think there is room for improvement. Status quo:

screen shot 2017-01-04 at 21 28 26

I recommend a stronger hierarchy between parents and children using size, indents and contrast:

screen shot 2017-01-04 at 21 28 00

(I forgot to make the header background color lighter, please imagine that, too ;-)

The same is true for the selected row in the layer list: Here higher contrast and maybe a bold font would help.

@lukasmartinelli
Copy link
Collaborator

It is at first sight indeed very difficult to orient yourself since everything looks the same.

@muesliq
Copy link
Collaborator Author

muesliq commented Jan 4, 2017

Yep. The perfect way would be to work on a really good color palette with good contrasts. The pragmatic way would be to simply use significantly lighter greys for headers, mouse-over and the active layer ;-) Plus bigger/stronger text for headers.

BTW, the fact that all CSS is inline and not in an external CSS file makes editing quite hard.

@lukasmartinelli
Copy link
Collaborator

I also found out that on bad contrast monitors the different shades of gray are probably too close to each other.

BTW, the fact that all CSS is inline and not in an external CSS file makes editing quite hard.

I am also no longer so convinced from the React inline style approach as of the beginning of the project.
It is great for components that are used in other projects - but if it is mostly components for the same app stylesheets make sense again.

@muesliq
Copy link
Collaborator Author

muesliq commented Jan 11, 2017

Replaced by #64.

@muesliq muesliq closed this as completed Jan 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants