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

feat(axis): alow user to hide tick lines while using culling #2480

Closed

Conversation

ImADrafter
Copy link

Issue

closes #2478

Details

Since I was the one who opened the issue I tried to figure out how could this feature should be implemented. Please let me know if either the feature is not necessary for the library or further changes are needed.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 90.193% when pulling 9540566 on ImADrafter:feat-hide-tick-lines-for-culling into c9a20b9 on naver:master.

@netil netil added the feature label Dec 30, 2021
@netil
Copy link
Member

netil commented Dec 30, 2021

Hi @ImADrafter, thanks for the contribution!

@netil netil closed this in aad8c45 Jan 3, 2022
@netil
Copy link
Member

netil commented Jan 3, 2022

Updated the option name from tick.culling.hideTickLines to tick.culling.lines, because the option already stays within "culling" and to simply the option name.

And also, the the newly added option should cover for y and y2 axes also.

axis: {
    [x | y | y2]: {
        culling: {
            max: 5,
            lines: false  // hide tick lines
        }

github-actions bot pushed a commit that referenced this pull request Jan 5, 2022
# [3.3.0-next.2](3.3.0-next.1...3.3.0-next.2) (2022-01-05)

### Bug Fixes

* **event:** fix touch event handling on arc ([d3d2e05](d3d2e05)), closes [#2477](#2477)
* **types:** Fix plugin's type definition ([f3690f9](f3690f9)), closes [#2483](#2483)

### Features

* **axis:** alow user to hide tick lines while using culling ([aad8c45](aad8c45)), closes [#2478](#2478) [#2480](#2480)
* **bar:** Intent to ship bar.indices.removeNull ([b16605d](b16605d)), closes [#1687](#1687)
github-actions bot pushed a commit that referenced this pull request Jan 14, 2022
# [3.3.0](3.2.2...3.3.0) (2022-01-14)

### Bug Fixes

* **api:** Ensure svg nodes to be removed from memory  ([f49ed83](f49ed83)), closes [#2489](#2489)
* **event:** fix touch event handling on arc ([d3d2e05](d3d2e05)), closes [#2477](#2477)
* **text:** Fix text position with candlestick type combination ([f84ab3e](f84ab3e)), closes [#2436](#2436)
* **tooltip:** fix candlestick tooltip display with xs option ([0278067](0278067)), closes [#2434](#2434)
* **types:** Fix axis types definition ([92fb033](92fb033)), closes [#2499](#2499)
* **types:** Fix plugin's type definition ([f3690f9](f3690f9)), closes [#2483](#2483)

### Features

* **axis:** alow user to hide tick lines while using culling ([aad8c45](aad8c45)), closes [#2478](#2478) [#2480](#2480)
* **bar:** add non zero based bar chart ([3588abe](3588abe)), closes [#2408](#2408) [#2438](#2438)
* **bar:** Implement stacking bar radius ([8f14d1a](8f14d1a)), closes [#2428](#2428)
* **bar:** Intent to ship bar.indices.removeNull ([b16605d](b16605d)), closes [#1687](#1687)
* **option:** Enhance padding to be removed completely ([2052a19](2052a19)), closes [#2367](#2367)
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.

[Feature request] Add an option for culling to also hide ticks, along with tick labels.
3 participants