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

"Container is too small" flashes briefly #3359

Closed
jimmyjones2 opened this issue Mar 16, 2015 · 2 comments
Closed

"Container is too small" flashes briefly #3359

jimmyjones2 opened this issue Mar 16, 2015 · 2 comments
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Feature:Vislib Vislib chart implementation Feature:Visualizations Generic visualization features (in case no more specific feature label is available)

Comments

@jimmyjones2
Copy link
Contributor

Either when you press F5, create a new search or change the index pattern. Only comes up for <1 second, then gets replaced by the time chart:
kbn-toosmall

@rashidkpc rashidkpc added the bug Fixes for quality problems that affect the customer experience label Mar 16, 2015
@rashidkpc rashidkpc changed the title Incorrect discover error message briefy shown "Container is too small" flashes briefly Mar 16, 2015
@rashidkpc rashidkpc added the Feature:Vislib Vislib chart implementation label Mar 16, 2015
@stormpython
Copy link
Contributor

@rashidkpc this happens because of a timing issue. The problem is that the visLib object is expecting a DOM element to be ready, but it is not, so the correct error is thrown. The chart is rendered before the DOM element is available. We cannot simply ignore the first error as this would cause some real errors not to be shown. We tried solving this issue before, but no appropriate solution was presented. I used an Angular $timeout which fixed the issue, but was rejected as a solution since it would be the same as using a setTimeout interval. But I wasn't really clear on why that was not a good solution.

In any case, this is a well known behavior, so I am not sure if the bug label is appropriate here.

@tbragin tbragin added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label Nov 9, 2016
@ppisljar ppisljar self-assigned this Nov 29, 2016
@ppisljar
Copy link
Member

ppisljar commented Dec 9, 2016

fixed by #9044

@ppisljar ppisljar closed this as completed Dec 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Vislib Vislib chart implementation Feature:Visualizations Generic visualization features (in case no more specific feature label is available)
Projects
None yet
Development

No branches or pull requests

5 participants