Skip to content

fix: popper scroll issue#712

Merged
nickofthyme merged 2 commits intoelastic:masterfrom
nickofthyme:fix/popper-scroll-issue
Jun 16, 2020
Merged

fix: popper scroll issue#712
nickofthyme merged 2 commits intoelastic:masterfrom
nickofthyme:fix/popper-scroll-issue

Conversation

@nickofthyme
Copy link
Copy Markdown
Collaborator

Summary

Fix scroll bar issue with popper js

@nickofthyme nickofthyme requested a review from markov00 June 16, 2020 14:13
@nickofthyme
Copy link
Copy Markdown
Collaborator Author

It looks like this has some very slight changes in the screenshots. Particularly with the box-shadow.

image

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 16, 2020

Codecov Report

Merging #712 into master will increase coverage by 0.35%.
The diff coverage is 73.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #712      +/-   ##
==========================================
+ Coverage   74.80%   75.16%   +0.35%     
==========================================
  Files         264      280      +16     
  Lines        8514     8842     +328     
  Branches     1709     1759      +50     
==========================================
+ Hits         6369     6646     +277     
- Misses       2093     2138      +45     
- Partials       52       58       +6     
Impacted Files Coverage Δ
...ypes/partition_chart/renderer/canvas/partition.tsx 32.39% <ø> (ø)
...chart_types/xy_chart/renderer/canvas/axes/title.ts 12.50% <0.00%> (ø)
src/components/portal/tooltip_portal.tsx 65.62% <ø> (ø)
src/utils/commons.ts 95.93% <ø> (-0.07%) ⬇️
src/state/chart_state.ts 87.09% <50.00%> (+0.21%) ⬆️
src/utils/logger.ts 54.54% <54.54%> (ø)
src/chart_types/xy_chart/domains/x_domain.ts 97.64% <71.42%> (-1.15%) ⬇️
src/chart_types/xy_chart/utils/series.ts 96.58% <88.88%> (-0.82%) ⬇️
src/chart_types/xy_chart/state/utils/utils.ts 92.95% <100.00%> (+0.50%) ⬆️
src/mocks/series/series_identifiers.ts 100.00% <0.00%> (ø)
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e26430...4cbab7f. Read the comment docs.

@nickofthyme
Copy link
Copy Markdown
Collaborator Author

nickofthyme commented Jun 16, 2020

I've also noticed that some screenshots appear to be slightly smaller in width. Notice the sliver of transparency on the left edge of the new screenshot. But yet the dimensions are identical. I saw this on #629 as well. Any ideas? Maybe it's just a github thing.

Image 2020-06-16 at 10 16 27 AM

Copy link
Copy Markdown
Collaborator

@markov00 markov00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nickofthyme nickofthyme merged commit 0c97c67 into elastic:master Jun 16, 2020
@nickofthyme nickofthyme deleted the fix/popper-scroll-issue branch June 16, 2020 15:45
markov00 pushed a commit that referenced this pull request Jun 16, 2020
## [19.5.1](v19.5.0...v19.5.1) (2020-06-16)

### Bug Fixes

* graceful scale fallbacks and warnings ([#704](#704)) ([ed49bbb](ed49bbb)), closes [#678](#678)
* **axis:** rotated label positioning ([#709](#709)) ([2e26430](2e26430)), closes [#673](#673)
* **tooltip:** popper scroll issue ([#712](#712)) ([0c97c67](0c97c67))
@markov00
Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 19.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@markov00 markov00 added the released Issue released publicly label Jun 16, 2020
AMoo-Miki pushed a commit to AMoo-Miki/OpenSearch-Dashboards that referenced this pull request Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Issue released publicly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants