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

Input Controls have wrong state when typing and hitting enter too fast throws error #22233

Closed
liza-mae opened this issue Aug 21, 2018 · 7 comments
Assignees
Labels
bug Fixes for quality problems that affect the customer experience failed-test A test failure on a tracked branch, potentially flaky-test Feature:Input Control Input controls visualization Team:Visualizations Visualization editors, elastic-charts and infrastructure test

Comments

@liza-mae
Copy link
Contributor

Kibana version: 6.4. BC6

Elasticsearch version: 6.4 BC6

Server OS version: WIndows

Browser version: Chrome

Browser OS version: Latest

Original install method (e.g. download page, yum, from source, etc.): staging

Describe the bug:
× fail: "visualize app input control visualization updateFiltersOnChange is false should replace existing filter pill(s) when new item is selected"
│ Error: expected true to equal false
│ at Assertion.assert (node_modules\expect.js\index.js:96:13)
│ at Assertion.be.Assertion.equal (node_modules\expect.js\index.js:216:10)
│ at Context.it (C:/Users/vagrant/Final/kibana-6.4/test/functional/apps/visualize/_input_control_vis.js:92:33)
│ at
│ at process._tickCallback (internal/process/next_tick.js:188:7)

@liza-mae
Copy link
Contributor Author

visualize app input control visualization updatefiltersonchange is false should replace existing filter pill_s_ when new item is selected

@liza-mae liza-mae added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label Aug 23, 2018
@stacey-gammon
Copy link
Contributor

This is a legit bug.

inputcontrolbug

cc @elastic/kibana-sharing

@stacey-gammon stacey-gammon added bug Fixes for quality problems that affect the customer experience :Sharing Feature:Input Control Input controls visualization and removed Feature:Visualizations Generic visualization features (in case no more specific feature label is available) labels Sep 9, 2018
@stacey-gammon
Copy link
Contributor

In addition, in trying to bypass the bug in the tests, ran into another issue. If you press the enter key too quickly while it's loading up you'll get an error Uncaught TypeError: Cannot read property 'click' of null (http://localhost:5620/bundles/vendors.bundle.js:187778)

@stacey-gammon stacey-gammon changed the title visualize app input control visualization updateFiltersOnChange is false should replace existing filter pill(s) when new item is selected Input Controls have wrong state when typing and hitting enter too fast throws error Sep 9, 2018
@nreese nreese self-assigned this Sep 9, 2018
@stacey-gammon
Copy link
Contributor

The problem with pressing enter I believe is an issue in EUI, but input controls has it's own issue.

when it searches, it flickers with all the options before being filtered:

ListControl.render options: [{"label":"css","value":"css"},{"label":"gif","value":"gif"},{"label":"jpg","value":"jpg"},{"label":"php","value":"php"},{"label":"png","value":"png"}]
list_control.js:84 ListControl.fetchOptions: c
list_control.js:89 ListControl.fetchOptions returned: c
list_control.js:148 ListControl.render options: [{"label":"css","value":"css"},{"label":"gif","value":"gif"},{"label":"jpg","value":"jpg"},{"label":"php","value":"php"},{"label":"png","value":"png"}]
list_control.js:148 ListControl.render options: [{"label":"css","value":"css"}]

That second to last render with all the options causes a flicker, which also throws off the tests since it tries to click the css that shows up then disappears.

@timroes timroes added Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed Team:Visualizations Visualization editors, elastic-charts and infrastructure :Sharing labels Sep 13, 2018
@liza-mae liza-mae added failed-test A test failure on a tracked branch, potentially flaky-test and removed flaky-failing-test labels Nov 1, 2018
@mbondyra mbondyra self-assigned this Jul 27, 2020
@kertal
Copy link
Member

kertal commented Jul 31, 2020

@mbondyra this can be closed, right?

@mbondyra
Copy link
Contributor

@kertal yeah, I will close it - planning on checking today if it in fact solved the issue and then closing it.

@mbondyra
Copy link
Contributor

mbondyra commented Aug 2, 2020

image

looks like the test is passing now, closing.

@mbondyra mbondyra closed this as completed Aug 2, 2020
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 failed-test A test failure on a tracked branch, potentially flaky-test Feature:Input Control Input controls visualization Team:Visualizations Visualization editors, elastic-charts and infrastructure test
Projects
None yet
Development

No branches or pull requests

6 participants