Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Make possible to disable the annoying "This dashboard is currently force refreshing" message #13242

Closed
ktecho opened this issue Feb 19, 2021 · 11 comments
Labels
enhancement:request Enhancement request submitted by anyone from the community

Comments

@ktecho
Copy link

ktecho commented Feb 19, 2021

Is your feature request related to a problem? Please describe.

With Superset 1.0, it seems a new toast has been added for when you have auto-refresh in a dashboard:

"This dashboard is currently force refreshing; the next force refresh will be in 10 seconds."

This is super-annoying because it distracts your attention from the changing dashboard. I think the "infinite / superset" animated symbol in each graph is enough. Besides, if you go to another tab in your browser, sometimes you can see more than one toast stacked.

Describe the solution you'd like

If you think it's a good idea to have the toast displaying each 10 seconds (maybe once at first would be fine...), please make it configurable so I can disable it from the UI or directly in the json config.

Describe alternatives you've considered

One alternative would be displaying the message just once when you open a dashboard.

@junlincc
Copy link
Member

junlincc commented Feb 19, 2021

if you go to another tab in your browser, sometimes you can see more than one toast stacked.

@ktecho Thanks for reporting. I sometimes find the toast massage taking away the focus and annoying too(as the PM😅). agree that one loading-infinity icon should be sufficient to indicate refresh activity.
@Steejay wdyt?

Screen.Recording.2021-02-19.at.3.10.52.PM.mov

@junlincc junlincc added need:design-review Requires input/approval from a Designer design-sys-revisit labels Feb 19, 2021
@Steejay
Copy link

Steejay commented Feb 22, 2021

hmm interesting. i think displaying the toast every 10 seconds is excessive. its like an alarm clock you cant turn off in the morning. how do you turn on auto refresh? is it through the Superset UI? If so, we can add some information in the location of that control via the use of help text (in which we could take away the toast completely for this use case). Therefore when the user turns on auto refresh theyll understand the result of this action in context.

Another potential alternative would be to include a "last updated" indicator somewhere on the dashboard.

@ktecho
Copy link
Author

ktecho commented Feb 22, 2021

Yes, I like @Steejay suggestion about including a "last updated" indicator. But even without that, you'll see that the dashboard is auto-refreshing because the widgets refreshes with the animated infinite symbol.

@Steejay
Copy link

Steejay commented Feb 22, 2021

@ktecho agreed. it would be good to consider the users who might experience the dashboard too. for the dashboard creator the auto refresh cadence might be well understood but for someone who only observes the dashboard, they might not have context on how often the dashboard is refreshing unless they're counting the intervals. are we willing to accept this potential ambiguity? if so the loading symbol might be enough as is. if not a "last updated" indicator could add context while still staying relatively subtle.

@ktecho
Copy link
Author

ktecho commented Feb 22, 2021

Yeah, totally agree.

@junlincc
Copy link
Member

junlincc commented Mar 31, 2021

we had a discussion about adding a status bar to show some basic info and dashboard interaction setting in the dashboard view, which could also include 'last refresh'. cc designer @mihir174

@ktecho can we agree that "last updated" indicator is still necessary for dashboard consumer, but not necessary in a form of the toast msg?

@ktecho
Copy link
Author

ktecho commented Mar 31, 2021

@junlincc Yeah, I think that would make it.

@junlincc junlincc added viz:dashboard:ui enhancement:request Enhancement request submitted by anyone from the community and removed need:design-review Requires input/approval from a Designer labels Mar 31, 2021
@Maximilian-Pichler
Copy link

+1

3 similar comments
@xallaume
Copy link

+1

@ga2arch
Copy link

ga2arch commented Aug 18, 2021

+1

@johhnydinh
Copy link

+1

@apache apache locked and limited conversation to collaborators Feb 2, 2022
@geido geido converted this issue into discussion #18451 Feb 2, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
enhancement:request Enhancement request submitted by anyone from the community
Projects
None yet
Development

No branches or pull requests

7 participants