-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
EPIC - Concept for updated app visual design #4514
Comments
Thanks for sharing this @corywatilo! I really like the overall aesthetics of this (though they do seem a tad to playful, if that makes sense), but wonder if right now is worth spending time on redoing the UI. That being said, I definitely think it's worth exploring further. Some specific comments.
|
Hey @corywatilo did we move away from this horizontal layout purposefully? I think this was a great use of the space. #4479 (comment) I really think it's important to have chart results and the events / filters in the same fold, I think it's difficult to do that when stacking the elements vertically. |
I personally greatly prefer the side-by-side layout, but this was changed for certain analysis views at @paolodamico's request due to the concern about long event/actions that take up a lot of horizontal space (and would otherwise wrap). It's probably worth a nice long exercise once we have a product designer on board who can put a lot more thought into this! |
Thanks for that context @corywatilo! @paolodamico I'd love to hear more about your thought process here and how we can optimize for some of the edge cases you're concerned about. I really feel strongly the right way to move forward is to optimize for the 80% and have these things in the same fold. Would love to think about how we tackle the 20%, maybe we can have a session this sprint to discuss? |
The above being said, |
Been thinking on this some more as well as playing around in other apps – I think most of the pain here would actually be solved by chart legends. I'm going to spend some time going deeper on this and putting some more thoughts together and will share with you all for feedback during this sprint. |
Sounds good @kpthatsme! We're also going to be working on #4594 this sprint so your input will be quite useful. |
I think we can close this considering that @clarkus is now heading the app's design and some big improvements have already been implemented. |
By no means is this a completed, fully fleshed out design, but I've been playing with some ideas and figured they were worth highlighting. Notes below.
Navigation
Querying workspace
Results/graph area
The text was updated successfully, but these errors were encountered: