[sql-lab] only scroll the result set table

[ascott]

before:

after:

fixes: #2231

todos:

• make table header fixed

@airbnb/superset-reviewers

[mistercrunch]

I think there was a related requirement to keep the headers on top. We may want to have a wrapper around the Table component that adds this as an extra prop "fixedHeaders"

[mistercrunch]

Oh now I'm noticing that the proportion of the page dedicated to viewing data is overall much smaller after the change. There should be a way to first scroll until the headers are at the top of the page, and then only scroll within the dataset.

[ascott]

with this change i'm trying to duplicate the airpal ui where only results are scrollable.

agreed that the table headers need to remain fixed before merging this in.

@mistercrunch i remember in an earlier prototype you had added a toggle to resize the results pane and sql editor, we could add that back in as well so users can make the results area larger if they want.

[ascott]

closing this in favor of two smaller PRs. one for the quick formatting win, and another next week for the refactored table component.