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

Format button and autocomplete checkbox should be always displayed #344

Closed
madalincm opened this issue Feb 27, 2018 · 8 comments
Closed

Comments

@madalincm
Copy link

Steps to Reproduce:

  1. Open new query page: https://pipeline-sql.stage.mozaws.net/queries/new
  2. Observe the layout.

Expected results:

The buttons are always displayed, the user has an easy access to all of them.

Actual results:

The user needs to hover inside the section where he writes the query for the Format button and autocomplete checkbox to be displayed

Notes/Issues:

Build identifier: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:58.0) Gecko/20100101 Firefox/58.0
Verified in FF58(Win7). Issue can be reproduced in STMO-stage
uichangesformatathover

@rafrombrc rafrombrc added this to the 15 milestone May 30, 2018
@alison985 alison985 self-assigned this Jun 15, 2018
@alison985
Copy link

This is reproducible on https://pipeline-sql.stage.mozaws.net/ today.

@alison985
Copy link

alison985 commented Jun 15, 2018

Upstream it's now a format button and there is no auto-complete checkbox. I'm submitting the change upstream ( getredash#2588 ) and we can port it back.

@alison985
Copy link

Related to getredash#2644

@kocsmy
Copy link

kocsmy commented Jul 8, 2018

I'm just wondering: is the auto-complete checkbox something that needs to be visible all the time (or more like a global settings that will be toggled once in a while)?

I'm wondering how could we find a place in the interface that can scale well with future improvements without over-populating the editor area.

If this will be a frequently used feature, it can go next to the new Parameters and Format buttons below the editor in getredash#2644

If it's a less important, maybe we'll need to introduce a new dropdown menu close to the editor where we can add things like this and even more later.

@alison985
Copy link

@jezdez FYI about the comment above.

@jezdez
Copy link

jezdez commented Jul 9, 2018

@alison985 Thank you!

@kocsmy Excellent points, scaling this high-density UI is tricky and is very likey to become more complicated the more the Redash query editor grows.

Especially with plans to add more support for frontend extensions (similar to the backend extensions) that our custom autocomplete checkbox could use, I would say the area next to the new Parameters and Format buttons like in getredash#2644 is hard to handle without adding an overflow technique like a dropdown. That said maybe we can table the scaling issue of that new area till a later date and just deal with the currently existing features.

So in short: +1 on your idea to implement it like in getredash#2644.

@rafrombrc rafrombrc modified the milestones: 15, 16 Jul 11, 2018
@rafrombrc rafrombrc modified the milestones: 16, 17 Aug 9, 2018
@jezdez
Copy link

jezdez commented Aug 9, 2018

getredash#2709 has moved the format button to the new area below the query textarea following getredash#2644.

I'm closing this now since porting the autocomplete toggle to upstream is tracked in #460.

@vcarciu
Copy link

vcarciu commented Sep 7, 2018

Verified as fixed :
format

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

6 participants