Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Dashboard not scroll up when user moving chart

See original GitHub issue

Dashboard not scroll up when user moving chart, or adding a new chart at the bottom of long dashboard.

Expected results

Be able to scroll so that chart can go to the place user wants.

Actual results

Doesn’t scroll.

How to reproduce the bug

open a large dashboard and start Edit mode
move chart from the bottom of page to dashboard top area
See dashbaord doesn’t scroll

Environment

(please complete the following information):

superset version: master

Screenshots

scroll-issue-1

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

This is regression - it was previously possible to scroll.

Issue Analytics

State:
Created 3 years ago
Reactions:1
Comments:10 (10 by maintainers)

Top GitHub Comments

2reactions

zuzana-vejcommented, Dec 4, 2020

Yes I just reproduced the 1st issue as well. Couldn’t repro the 2nd one today either (so I think we should just close off this bug once the 1st issue is fixed). The layout of my dashboard is here in a screenshot. It has 3 rows, and the top row still has space for another chart. I am trying to move the bottom chart to the top and it won’t move.

Screen Shot 2020-12-04 at 1 28 50 PM

1reaction

kkucharccommented, Dec 15, 2020

I had to move to another issue due to some priority change. To sum up what I managed to discover:

General issue: the scroll area is hidden under the dashboard’s header (z-index and position=relative of the header disables scroll). The best thing here would be to somehow move detecting scroll to top level, but I am not sure if it’s possible. The other option is to use dnd component and detect that we are hovering header and then scroll top on hover.
Second issue: some charts and tables have scroll when the window is too narrow. When we drag an element, the preview of dragged element is as wide as table/chart inside this element. After my debugging nothing seemed helping: width: 100% or overflow: hidden for divs, svg, table. What I may suggest is to disable scroll while editing or pass width as HTML attribute directly to svg or table.