-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
[dashboard v2] misc UI fixes, undo redo bug fix #5293
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5293 +/- ##
=========================================
- Coverage 61.3% 61.3% -0.01%
=========================================
Files 368 368
Lines 23453 23455 +2
Branches 2713 2715 +2
=========================================
+ Hits 14378 14379 +1
- Misses 9063 9064 +1
Partials 12 12
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners (cherry picked from commit 05a39b3)
…order (#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners
…order (apache#5293) * fix sqllab <Loading /> css, fix double AddSliceCard margin and drag border * [dashboard v2] improve cached visual indicator, add last fetched messaging * [dashboard v2] fix ctrl + cmd + z + UndoRedoKeylisteners
This PR fixes the following UI issues spotted in the new dashboard builder:
adds a check for
form_data
in theChart
component container to prevent throwing ifchart.form_data
is undefinedfixes a bug where our undo/redo event listeners considered 'z' a control key thus preventing users from adding a
z
to headers + slice headers 😬removes red from cached data visual indicator and adds a label for the last-fetched time
before
after
fixes css for SQL lab
<Loading />
instancebefore
after
refactors
AddSliceCard
styles to remove double marginbefore
after
@graceguo-supercat @john-bodley