[GitHub] [incubator-superset] Steejay commented on pull request #9794: feat: implement create view as functionality in Sqllab

[GitBox <gi...@apache.org>]

Steejay commented on pull request #9794:
URL: https://github.com/apache/incubator-superset/pull/9794#issuecomment-645640153


   Here is what i think might be a better experience as a short term solution:
   • i think theres an opportunity to consolidate the CTAS and CVAS buttons > can we consider one button "Save Table" ?
   • Once "Save Table" is selected a modal will pop up that allows the user to input a table name and • select between two options: CVAS or CTAS (radio button)
   • (see 'Save Query' modal)
   
   My concern is that there are a lot of save type buttons (save query, CTAS, CVAS) in a zone that has limited real estate especially in smaller windows. Consolidating similar functions has the opportunity to empower users to make decisions quicker by minimizing options. 
   
   If strong tech constraints exist that dont allow for the suggestions here. I think that this current layout is still worth considering. 


----------------------------------------------------------------
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