[GitHub] [superset] justinpark commented on pull request #24787: feat(sqllab): Show sql in the current result

["justinpark (via GitHub)" <gi...@apache.org>]

justinpark commented on PR #24787:
URL: https://github.com/apache/superset/pull/24787#issuecomment-1652360214

   > and the last one - this is more of a question - I'm wondering if instead of showing a pop up with the sql query, would it be more useful to scroll and highlight the query in the sql query area at the top? 
   
   @kasiazjc this is a good idea but it might be broken if a user modify the sql content which is not same from the ran query.
   
   


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

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

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