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

fix(api): fix tooltip work for .load() with xs key #3203

Merged
merged 1 commit into from
May 3, 2023

Conversation

netil
Copy link
Member

@netil netil commented May 3, 2023

Issue

#3194

Details

when data initializes with empty data, do not initialize eventRect and initializes when data is loaded dynamically.

May-03-2023 16-11-28

when data initializes with empty data, do not initialize eventRect
and initializes when data is loaded dynamicallay.

Ref naver#3194
@netil netil added the bug label May 3, 2023
@netil netil self-assigned this May 3, 2023
@coveralls
Copy link

Coverage Status

Coverage: 90.71% (+0.05%) from 90.664% when pulling 55af606 on netil:xs#3194 into cf82289 on naver:master.

@netil netil merged commit 13b9da7 into naver:master May 3, 2023
10 checks passed
@netil netil deleted the xs#3194 branch May 3, 2023 07:21
netil pushed a commit to netil/billboard.js that referenced this pull request May 18, 2023
Fix the side effect caused by naver#3203.
Make sure to set proper class name for eventRect element.

Ref 3218
netil added a commit that referenced this pull request May 18, 2023
Fix the side effect caused by #3203.
Make sure to set proper class name for eventRect element.

Fix #3218
netil pushed a commit that referenced this pull request May 18, 2023
Fix the side effect caused by #3203.
Make sure to set proper class name for eventRect element.

Ref #3218
github-actions bot pushed a commit that referenced this pull request May 18, 2023
## [3.8.2](3.8.1...3.8.2) (2023-05-18)

### Bug Fixes

* **api:** fix classname setting for  eventRect element ([0a2fbde](0a2fbde)), closes [#3203](#3203) [#3218](#3218)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants