-
Notifications
You must be signed in to change notification settings - Fork 14.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(dashboard): Fix scrolling on "View as table" modal #21282
Conversation
Codecov Report
@@ Coverage Diff @@
## master #21282 +/- ##
=======================================
Coverage 66.47% 66.47%
=======================================
Files 1789 1789
Lines 68376 68377 +1
Branches 7275 7275
=======================================
+ Hits 45450 45451 +1
Misses 21051 21051
Partials 1875 1875
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
superset-frontend/src/explore/components/DataTablesPane/components/ResultsPaneOnDashboard.tsx
Outdated
Show resolved
Hide resolved
@codyml in the video "After, single query" I can see that the results are showing up behind the upper part of the header when scrolling. Is that something that we can get fixed on this PR? |
Oops yes! |
@geido Hmm, so I looked at this in my local dev env and I think it might just be a scaling effect caused by me having my Chrome viewport set to 75% when I recorded those videos. When I set it to 100% it doesn't happen. Would you mind spinning up an ephemeral env and letting me know if you see it happening there at normal resolution? |
5815427
to
54a059d
Compare
/testenv up |
@zhaoyongjie Ephemeral environment spinning up at http://34.217.133.154:8080. Credentials are |
Ephemeral environment shutdown and build artifacts deleted. |
SUMMARY
In the "View as table" modal, accessed by clicking the three dots on a dashboard chart, results scroll behind/with the header and pagination. This PR fixes layout so header and pagination stay in place, for both the single-query and multi-query version of the modal.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before, single query:
Screen.Recording.2022-08-31.at.3.54.20.PM.mov
Before, multiple query:
Screen.Recording.2022-08-31.at.3.54.50.PM.mov
After, single query:
Screen.Recording.2022-08-31.at.3.55.21.PM.mov
After, multiple query:
Screen.Recording.2022-08-31.at.3.55.37.PM.mov
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION