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.

Sign up for GitHub

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

Jump to bottom

Bad tooltip placement with left/top legend #769

Closed
nickofthyme opened this issue Jul 29, 2020 · 1 comment · Fixed by #771
Closed

Bad tooltip placement with left/top legend #769

nickofthyme opened this issue Jul 29, 2020 · 1 comment · Fixed by #771
Assignees
@nickofthyme
Labels
bug Something isn't working released Issue released publicly :tooltip Related to hover tooltip

Comments

@nickofthyme
Copy link
Collaborator

nickofthyme commented Jul 29, 2020 •
edited
Loading

Describe the bug
When setting legendPosition to 'left' or 'top' the tooltip placement is not accurate. All other positions are correctly placed.

To Reproduce
Steps to reproduce the behavior:

  1. Run master locally
  2. Go to http://localhost:9001/?path=/story/legend--left
  3. Hover over any series

Expected behaviour
Correct placement.

Screenshots
Screen Recording 2020-07-30 at 11 35 AM

Version (please complete the following information):

  • Elastic Charts: as of at least 19.9.0
@nickofthyme nickofthyme added bug Something isn't working :tooltip Related to hover tooltip labels Jul 29, 2020
@nickofthyme nickofthyme self-assigned this Jul 29, 2020
@nickofthyme nickofthyme changed the title Bad tooltip placement with left side legend Bad tooltip placement with left/top legend Jul 30, 2020
@nickofthyme nickofthyme mentioned this issue Jul 30, 2020
Merged
4 tasks
nickofthyme added a commit that referenced this issue Aug 3, 2020
@nickofthyme
fix(tooltip): placement with left/top legends and single bars (#771)
e576b26 
fixes #769 and #770
markov00 pushed a commit that referenced this issue Aug 10, 2020
@semantic-release-bot
chore(release): 21.0.0 [skip ci]
e7a8a30 
# [21.0.0](v20.0.2...v21.0.0) (2020-08-10)

### Bug Fixes

* update dep vulnerabilities, minimist and kind-of ([#763](#763)) ([4455281](4455281))
* **legend:** fix color anchor, add action context, fix action padding ([#774](#774)) ([4590a22](4590a22))
* **tooltip:** placement with left/top legends and single bars ([#771](#771)) ([e576b26](e576b26)), closes [#769](#769) [#770](#770)

### Features

* streamgraph and fit functions on stacked charts ([#751](#751)) ([268fcc0](268fcc0)), closes [#766](#766) [#715](#715) [#450](#450)

### BREAKING CHANGES

* the first parameter of `PointStyleAccessor` and `BarStyleAccessor` callbacks is changed from `RawDataSeriesDatum` to `DataSeriesDatum`. `stackAsPercentage` prop is replaced by `stackMode` that accept one `StackMode`.
@markov00
Copy link
Member

🎉 This issue has been resolved in version 21.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@markov00 markov00 added the released Issue released publicly label Aug 10, 2020
AMoo-Miki pushed a commit to AMoo-Miki/OpenSearch-Dashboards that referenced this issue Feb 10, 2022
@semantic-release-bot
chore(release): 21.0.0 [skip ci]
5c68eec 
# [21.0.0](elastic/elastic-charts@v20.0.2...v21.0.0) (2020-08-10)

### Bug Fixes

* update dep vulnerabilities, minimist and kind-of ([opensearch-project#763](elastic/elastic-charts#763)) ([843554f](elastic/elastic-charts@843554f))
* **legend:** fix color anchor, add action context, fix action padding ([opensearch-project#774](elastic/elastic-charts#774)) ([262f8d2](elastic/elastic-charts@262f8d2))
* **tooltip:** placement with left/top legends and single bars ([opensearch-project#771](elastic/elastic-charts#771)) ([75533b1](elastic/elastic-charts@75533b1)), closes [opensearch-project#769](elastic/elastic-charts#769) [opensearch-project#770](elastic/elastic-charts#770)

### Features

* streamgraph and fit functions on stacked charts ([opensearch-project#751](elastic/elastic-charts#751)) ([6f6a8cb](elastic/elastic-charts@6f6a8cb)), closes [opensearch-project#766](elastic/elastic-charts#766) [opensearch-project#715](elastic/elastic-charts#715) [opensearch-project#450](elastic/elastic-charts#450)

### BREAKING CHANGES

* the first parameter of `PointStyleAccessor` and `BarStyleAccessor` callbacks is changed from `RawDataSeriesDatum` to `DataSeriesDatum`. `stackAsPercentage` prop is replaced by `stackMode` that accept one `StackMode`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nickofthyme nickofthyme

Labels
bug Something isn't working released Issue released publicly :tooltip Related to hover tooltip
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(tooltip): placement with left/top legends and single bars nickofthyme/elastic-charts
2 participants
@markov00 @nickofthyme