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

Fix Flash of Unpopulated Graphs on Refresh #967

Closed
wants to merge 0 commits into from

Conversation

timraymond
Copy link
Contributor

@timraymond timraymond commented Mar 6, 2017

  • CHANGELOG.md updated
  • Rebased/mergable
  • Tests pass
  • Sign CLA (if not already signed)

Connect #915

If a previous fetch of query results was unsuccessful, a "No Results"
message is displayed by the AutoRefresh component. When it went to fetch
results again, this would briefly be replaced by its composed compoenent
which would be instructed to show its "loading" treatment. Since "No
Results" is usually not a situation that would spontaneously change
without user intervention, this patch stores whether the previous fetch
was successful, and hides the loading treatment during subsequent
fetches if it wasn't.

@goller
Copy link
Contributor

goller commented Mar 9, 2017

Hey @timraymond I think this just needs its conflicts fixed and it looks ready to merge.

@121watts
Copy link
Contributor

121watts commented Mar 9, 2017

@timraymond lets get this merged in. Just fix the conflict and merge.

@timraymond timraymond closed this Mar 9, 2017
@timraymond timraymond force-pushed the bugfix/tr-flash-unpopulated-graphs branch from 843957b to d1102be Compare March 9, 2017 18:02
@timraymond timraymond deleted the bugfix/tr-flash-unpopulated-graphs branch March 9, 2017 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants