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

Vislib/Dashboard - adding/removing filters causes visualizations not to re-draw #2255

Closed
w33ble opened this issue Dec 11, 2014 · 3 comments · Fixed by #2258 or #2280
Closed

Vislib/Dashboard - adding/removing filters causes visualizations not to re-draw #2255

w33ble opened this issue Dec 11, 2014 · 3 comments · Fixed by #2258 or #2280
Assignees
Labels
blocker bug Fixes for quality problems that affect the customer experience PR sent

Comments

@w33ble
Copy link
Contributor

w33ble commented Dec 11, 2014

Repro:

  • Create a dashboard with a bunch of visualizations on it
  • Add a filter that (still contains data)

Line, histogram, and I assume area charts don't render correctly. Removing the filter does not help. You see the legends, but there are no graphs.

Refreshing the page brings them back.

2014-12-11 11_28_13

@w33ble w33ble added bug Fixes for quality problems that affect the customer experience blocker labels Dec 11, 2014
@w33ble
Copy link
Contributor Author

w33ble commented Dec 11, 2014

There's a related issue here as well - when you open the spy on the vis to make sure there is actually data, you get a fatal error when you close the vis.

2014-12-11 11_33_38

screenshot 2014-12-11 11 33 30

EventEmitter: Cannot read property 'getPropertyValue' of null
TypeError: Cannot read property 'getPropertyValue' of null
    at Array.d3_selectionPrototype.style (http://localhost:8000/bower_components/d3/d3.js:666:70)
    at HTMLDivElement.<anonymous> (http://localhost:8000/components/vislib/lib/x_axis.js:447:55)
    at http://localhost:8000/bower_components/d3/d3.js:884:16
    at d3_selection_each (http://localhost:8000/bower_components/d3/d3.js:890:30)
    at Array.d3_selectionPrototype.each (http://localhost:8000/bower_components/d3/d3.js:883:12)
    at XAxis.updateXaxisHeight (http://localhost:8000/components/vislib/lib/x_axis.js:439:17)
    at Array.XAxis.filterOrRotate (http://localhost:8000/components/vislib/lib/x_axis.js:218:14)
    at Array.d3_selectionPrototype.call (http://localhost:8000/bower_components/d3/d3.js:897:14)
    at Array.XAxis.draw (http://localhost:8000/components/vislib/lib/x_axis.js:189:19)
    at Array.d3_selectionPrototype.call (http://localhost:8000/bower_components/d3/d3.js:897:14)

@jthomassie jthomassie self-assigned this Dec 11, 2014
stormpython added a commit to stormpython/kibana that referenced this issue Dec 11, 2014
w33ble added a commit that referenced this issue Dec 11, 2014
Allow filtering on multiple visualizations in the dashboard.
@rashidkpc rashidkpc reopened this Dec 11, 2014
@rashidkpc
Copy link
Contributor

Reopening as this is still happening. It appears to have something todo with handling of empty buckets. Here I'm creating a filter that would end up with hits, but because the visualization deals with the phpmemory field, and not all docs have a phpmemory field, the aggregation is empty. The visualization is rendered as a blank canvas, and upon resize, fatals.

2014-12-11 16_36_16

@spalger
Copy link
Contributor

spalger commented Dec 12, 2014

@rashidkpc can't seem to reproduce this.

stormpython added a commit to stormpython/kibana that referenced this issue Dec 12, 2014
rashidkpc added a commit that referenced this issue Dec 12, 2014
Allow filtering on multiple visualizations in the dashboard.
markov00 added a commit that referenced this issue Dec 21, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@elastic/charts](https://togithub.com/elastic/elastic-charts) |
[`61.0.3` ->
`61.2.0`](https://renovatebot.com/diffs/npm/@elastic%2fcharts/61.0.3/61.2.0)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@elastic%2fcharts/61.2.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@elastic%2fcharts/61.2.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@elastic%2fcharts/61.0.3/61.2.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@elastic%2fcharts/61.0.3/61.2.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>elastic/elastic-charts (@&#8203;elastic/charts)</summary>

###
[61.2.0](elastic/elastic-charts@v61.1.0...v61.2.0)
(2023-12-19)


##### Bug Fixes

* **deps:** update dependency @elastic/eui to ^91.1.0
([#2267](elastic/elastic-charts#2267))
([308e974](elastic/elastic-charts@308e974))
* **deps:** update dependency @elastic/eui to ^91.2.0
([#2268](elastic/elastic-charts#2268))
([29cdcb3](elastic/elastic-charts@29cdcb3))
* **metric:** background colors and sparkline rendering
([#2255](elastic/elastic-charts#2255))
([5abddfc](elastic/elastic-charts@5abddfc))
* **partition:** rendering with small radius
([#2273](elastic/elastic-charts#2273))
([95a8537](elastic/elastic-charts@95a8537))
* **partition:** zero value sectors cause max stack call
([#2260](elastic/elastic-charts#2260))
([4b30db7](elastic/elastic-charts@4b30db7))
* **theme:** legacy margins
([#2262](elastic/elastic-charts#2262))
([299c869](elastic/elastic-charts@299c869))


##### Features

* increase tooltip width to 500px and truncate items to 2 lines
([#2261](elastic/elastic-charts#2261))
([afdef1c](elastic/elastic-charts@afdef1c))

###
[`v61.1.0`](https://togithub.com/elastic/elastic-charts/blob/HEAD/CHANGELOG.md#6110-2023-11-20)

[Compare
Source](https://togithub.com/elastic/elastic-charts/compare/v61.0.3...v61.1.0)

##### Bug Fixes

- **deps:** update dependency
[@&#8203;elastic/eui](https://togithub.com/elastic/eui) to v91
([#&#8203;2233](https://togithub.com/elastic/elastic-charts/issues/2233))
([e89f623](https://togithub.com/elastic/elastic-charts/commit/e89f623792312c4f6b609ebb975de0800f3c297e))
- **metric:** add option to set empty cell background color
([#&#8203;2247](https://togithub.com/elastic/elastic-charts/issues/2247))
([6a9fb86](https://togithub.com/elastic/elastic-charts/commit/6a9fb86bee5212a47060c5070f260961097014b4))
- **metric:** background color for bar with interactions
([#&#8203;2248](https://togithub.com/elastic/elastic-charts/issues/2248))
([dcb56fa](https://togithub.com/elastic/elastic-charts/commit/dcb56fa08540631a9b3b0e588352ee6daf3d54a0))

##### Features

- **bullet:** improve header layout and positioning
([#&#8203;2243](https://togithub.com/elastic/elastic-charts/issues/2243))
([b3a95d2](https://togithub.com/elastic/elastic-charts/commit/b3a95d24fb02690ca6599622352c743c04624690))
- **bullet:** new design style and implementation
([#&#8203;2156](https://togithub.com/elastic/elastic-charts/issues/2156))
([fca6cdd](https://togithub.com/elastic/elastic-charts/commit/fca6cdd5bc34a65c0792dbab7d756404bf43501b))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/elastic/kibana).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44Ny4yIiwidXBkYXRlZEluVmVyIjoiMzcuMTAzLjEiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Marco Vettorello <marco.vettorello@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker bug Fixes for quality problems that affect the customer experience PR sent
Projects
None yet
5 participants