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(interaction): Fix unset focused state #2928

Merged
merged 1 commit into from
Oct 28, 2022
Merged

Conversation

netil
Copy link
Member

@netil netil commented Oct 28, 2022

Issue

#2927

Details

Before setting focused state, make sure to remove possible previous focused state by removing "expanded" class.

focused-state

Before setting focused state, make sure to remove possible
previous focused state by removing "_expanded_" class.

Ref naver#2927
@netil netil added the bug label Oct 28, 2022
@netil netil self-assigned this Oct 28, 2022
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 90.734% when pulling 5002a82 on netil:expand#2927 into 3d43066 on naver:master.

@netil netil merged commit 65de466 into naver:master Oct 28, 2022
netil pushed a commit to netil/billboard.js that referenced this pull request Jan 9, 2023
…tiple=false

- Fix selection to be one at a time when selection.multiple=false is set
- Refactor .select()/.unselect() API
- Fix side-effect caused by naver#2928

Ref naver#3013
netil added a commit that referenced this pull request Jan 9, 2023
…tiple=false

- Fix selection to be one at a time when selection.multiple=false is set
- Refactor .select()/.unselect() API
- Fix side-effect caused by #2928

Fix #3013
github-actions bot pushed a commit that referenced this pull request Jan 9, 2023
# [3.7.0-next.3](3.7.0-next.2...3.7.0-next.3) (2023-01-09)

### Bug Fixes

* **plugin:** Fix stanford scale to accept 0 as min value ([02c4270](02c4270)), closes [#2999](#2999)
* **selection:** Fix selection w/point.focus.only=true & selection.multiple=false ([af93a8a](af93a8a)), closes [#2928](#2928) [#3013](#3013)
github-actions bot pushed a commit that referenced this pull request Jan 11, 2023
# [3.7.0](3.6.3...3.7.0) (2023-01-11)

### Bug Fixes

* **axis:** Fix axis label overlap when has no data ([643357b](643357b)), closes [#2974](#2974)
* **axis:** fix axis.y/y2.tick.format context error ([54ee419](54ee419)), closes [#2976](#2976)
* **domain:** Fix wrong domain range setting ([b55b153](b55b153)), closes [#2962](#2962)
* **options:** Fix data.regions path rendering for category axis ([249d0ac](249d0ac)), closes [#2950](#2950)
* **plugin:** Fix stanford scale to accept 0 as min value ([02c4270](02c4270)), closes [#2999](#2999)
* **selection:** Fix selection w/point.focus.only=true & selection.multiple=false ([af93a8a](af93a8a)), closes [#2928](#2928) [#3013](#3013)
* **types:** Add missing param definition for gauge.label.format ([d663532](d663532)), closes [#2946](#2946)
* **types:** Fix to referenced but missing d3 types ([0b57454](0b57454)), closes [#2954](#2954) [#2961](#2961)
* **types:** Support dual module type.d in TS >= 4.7 ([143e491](143e491)), closes [#2952](#2952) [#2953](#2953)

### Features

* **api:** Enhance .export() to preserve fontStyle ([0809ae7](0809ae7)), closes [#2892](#2892)
* **arc:** Intent to ship arc.cornerRadius ([e8320cf](e8320cf)), closes [#2936](#2936)
* **tooltip:** Enhance passing ratio value for stacked area/bar ([9e7ff01](9e7ff01)), closes [#2921](#2921)
* **treemap:** Intent to ship treemap type ([9809ef1](9809ef1)), closes [#123](#123) [#3002](#3002) [#3004](#3004) [#3006](#3006) [#3007](#3007) [#3008](#3008)
@netil netil deleted the expand#2927 branch September 8, 2023 06:18
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