You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2020/11/05 22:30:41 UTC

[GitHub] [incubator-superset] betodealmeida opened a new issue #11586: Improve selection of LIMIT in SQL Lab

betodealmeida opened a new issue #11586:
URL: https://github.com/apache/incubator-superset/issues/11586


   **Is your feature request related to a problem? Please describe.**
   
   In SQL Lab, to change the current limit the user needs to:
   
   1. Click the LIMIT button
   2. Click the input field
   3. Delete the current value
   4. Type in the new value
   5. Press OK
   
   This is a lot.
   
   **Describe the solution you'd like**
   
   In SQL Lab, after clicking the "LIMIT" button the input element in the dialog that pops should be focused and the current value selected. Pressing enter should apply and close the dialog, equivalent to clicking OK.
   
   1. Click LIMIT button
   2. Type in the new value
   3. Press ENTER
   
   **Describe alternatives you've considered**
   
   N/A
   
   **Additional context**
   
   N/A


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [incubator-superset] mistercrunch commented on issue #11586: Improve selection of LIMIT in SQL Lab

Posted by GitBox <gi...@apache.org>.
mistercrunch commented on issue #11586:
URL: https://github.com/apache/incubator-superset/issues/11586#issuecomment-723710469


   We have an internal design to improve the SQL Lab toolbar, and that includes improvement to the LIMIT control, it'd be great if we can share the design here in the open.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [incubator-superset] Giphyn-nehboh-5qonhu commented on issue #11586: Improve selection of LIMIT in SQL Lab

Posted by GitBox <gi...@apache.org>.
Giphyn-nehboh-5qonhu commented on issue #11586:
URL: https://github.com/apache/incubator-superset/issues/11586#issuecomment-722737949


   https://t.me/Hos_144


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [incubator-superset] issue-label-bot[bot] commented on issue #11586: Improve selection of LIMIT in SQL Lab

Posted by GitBox <gi...@apache.org>.
issue-label-bot[bot] commented on issue #11586:
URL: https://github.com/apache/incubator-superset/issues/11586#issuecomment-722683378


   Issue-Label Bot is automatically applying the label `#enhancement` to this issue, with a confidence of 0.97. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback! 
   
    Links: [app homepage](https://github.com/marketplace/issue-label-bot), [dashboard](https://mlbot.net/data/apache/incubator-superset) and [code](https://github.com/hamelsmu/MLapp) for this bot.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [incubator-superset] Steejay commented on issue #11586: Improve selection of LIMIT in SQL Lab

Posted by GitBox <gi...@apache.org>.
Steejay commented on issue #11586:
URL: https://github.com/apache/incubator-superset/issues/11586#issuecomment-724872046


   hey @betodealmeida ! We're exploring this new limit design shown in the screenshot here. I agree that the current limit interaction should be simplified. We also saw an opportunity to eliminate some of the annoying error states a user might encounter (empty text field and integer over max limit). We believe that this dropdown of pre defined limits can help assuage some of these pain points. If a user wants to define a custom limit they still have the ability to do so in SQL Editor. 
   
   
   ![Frame 213 (1)](https://user-images.githubusercontent.com/60786102/98712579-5588bf80-233b-11eb-948d-d5ad37b00908.jpg)
   
   cc @yousoph @mistercrunch 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org