Skip to content
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] Scrolling table viz overlaps next chart #18223

Closed
3 tasks done
zuzana-vej opened this issue Jan 29, 2022 · 3 comments
Closed
3 tasks done

[dashboard] Scrolling table viz overlaps next chart #18223

zuzana-vej opened this issue Jan 29, 2022 · 3 comments
Labels
#bug:cosmetic Cosmetic/layout/design tweak needed #bug Bug report preset:cares Preset cares about this issue validation:validated A committer has validated / submitted the issue or it was reported by multiple users

Comments

@zuzana-vej
Copy link
Contributor

If you have a chart on dashboard, which isn't fully displayed - e.g. need scrolling to see other records, it overlaps the next item on the dashboard (eg. a title, or another chart). Upon some investigation, we determine it is specifically a bug with chart that have descriptions. If you toggle the chart description off and then on again, all works well, but when you refresh the bug is back.

In case the table has pagination, the pagination bar is overflowing - see screenshots.

There is a workaround which minimizes the impact when user adds empty row below the chart, but then they see the grey text "empty row" so it's still not ideal. (instead of scrolling over chart or title, it's over the grey text"empty row").

How to reproduce the bug

described above

Screenshots

without toggle (correct)
Screen Shot 2022-01-28 at 3 59 58 PM

(with toggle description - overflow)
Screen Shot 2022-01-28 at 3 59 49 PM

Another example of the same issue:
Screen Shot 2022-01-28 at 4 01 25 PM

Environment

latest master

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.
@zuzana-vej zuzana-vej added #bug Bug report viz:dashboard:ui #bug:cosmetic Cosmetic/layout/design tweak needed labels Jan 29, 2022
@geido geido added validation:validated A committer has validated / submitted the issue or it was reported by multiple users preset:cares Preset cares about this issue labels Feb 1, 2022
@geido
Copy link
Member

geido commented Feb 14, 2022

Hello @zuzana-vej I was looking back into this issue and I cannot reproduce it right now. Could you please try again with the latest master? If you still encounter the issue please provide more information such as browser and version used. Thanks!

@zuzana-vej
Copy link
Contributor Author

Here are exact steps to repro this:
Chart has description. If you just toggle the description it will be ok and it will shift the title of the next section down. If you leave the description toggled, and then refresh the page, it will load the chart with description toggled and the bottom of the table will overlap the next section.
If you put white background to the header which is below the chart, it will not overlap, but it will have half of the last row unreadable which is even worse.

Step 1: toggle chart description --> no problem.
Step 1b: if you untoggle & toggle --> no problem.
Screen Shot 2022-02-22 at 4 03 12 PM

Step 2: Keep toggled and refresh page (ctrl+R). Overlap - --> problem with overlap.
Screen Shot 2022-02-22 at 4 01 41 PM

Browser Google Chrome Version 98.0.4758.102 (Official Build) (x86_64)
Latest master - released from master on 02/17

@serenajiang
Copy link
Contributor

Thanks for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug:cosmetic Cosmetic/layout/design tweak needed #bug Bug report preset:cares Preset cares about this issue validation:validated A committer has validated / submitted the issue or it was reported by multiple users
Projects
None yet
Development

No branches or pull requests

3 participants