-
Notifications
You must be signed in to change notification settings - Fork 12.1k
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
UX: Query editors general layout improvement #3900
Comments
ping @mattttt |
Not sure if it's just because I'm used to seeing the eye and query letter Taking another look at idea 1, I do like the visual grouping you've added. I feel like the letter and eye need to live next to each other. Separating On Sun, Jan 31, 2016 at 12:03 PM Torkel Ödegaard notifications@github.com
|
@bulletfactory yea, I agree, the letter and eye work better together and to the left, it's just sad that they require an indentation of the lower rows |
I dont think the issue is really about the unused space, but the perceived waste of space. Im thinking we can detach the Letter and Eye from the tightform somehow, making the query editor itself feel very tight/efficient, and that unused space will disappear to the eye. I dont like moving the letter to the right, as it feels like a very important identifier (instead of an attribute of the query). Also, agreed with @bulletfactory's point on keep those connected. Will have to try a few mockups to see if it can work the way I'm thinking. |
I think 3 works best as well. On Tue, Feb 2, 2016 at 4:16 PM Matt Toback notifications@github.com wrote:
|
@mattttt yea, the collapsed state is a great idea, hm.. damit that could be important to nail down before 3.0 beta release as it could affect the query editor shared component, I kind of like the middle option best of your previous comment (with the letter and eye hanging out). Something else to think about is the blue color and font, never been truly happy with the blue color and boldness (too bold), but never found a good alternative that fills the same function |
@torkelo / @bulletfactory: Would this be a job for....semi-bold?!?!? |
@mattttt that was my thinking as well :) |
think I will close this, maybe create new one issue to discuss what the query editors could look like inspired by / or using the new forms styles |
Testing how the new form styles would look like for InfluxDB Query editor. Overall works pretty well, but looks a strange with all dark areas sort of creating a tetris like profile. Think the raw query mode look could work as a general look for a collapsed state of a query (even for data sources that dont have a string based raw query, the collapsed mode could just show a single line text representation of the query that tries to capture the gist of the query) |
Think there is some improvement needed in how queries are layout out.
Problem, the current layout of the letter and eye to the right causes some empty unused space on the rows below.
Same problem for elastic search and cloudwatch and OpenTSDB
Idea 1) put query letter and eye to the right,
Idea 2) Put letter to the left outside the rows, move eye to the right
(no image of this idea).
On thing that became apparent was that when I moved the letter and eye to the right spacing between the queries becomes necessary as the queries blend into each other more.
The text was updated successfully, but these errors were encountered: