SQL Lab: CTRL+R and CTRL+ENTER bypass disabled RUN button

[betodealmeida]

In SQL Lab, if the text area is empty the "RUN" button is disabled, but the user can still press the key shortcut to run the empty query. This results in an error.

Expected results

Shortcut keys should be ignored.

Actual results

Query runs.

Screenshots

How to reproduce the bug

1. Go to SQL Lab
2. Delete everything in the text area (including SELECT ...)
3. Press CTRL+R
4. See error

Environment

(please complete the following information):

• superset version: master
• python version: 3.8
• node.js version: 1.14.11

Checklist

Make sure to follow these steps before submitting your issue - thank you!

• I have checked the superset logs for python stacktraces and included it here as text if there are any.
• I have reproduced the issue with at least the latest released version of superset.
• I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context

N/A

[issue-label-bot]

Issue-Label Bot is automatically applying the label #bug to this issue, with a confidence of 0.92. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.