Skip to content
This repository was archived by the owner on Mar 7, 2026. It is now read-only.

docs: add API documentation for includeHidden axis prop#223

Merged
ckifer merged 2 commits into
recharts:masterfrom
neefrehman:feat/static-domain
Mar 15, 2023
Merged

docs: add API documentation for includeHidden axis prop#223
ckifer merged 2 commits into
recharts:masterfrom
neefrehman:feat/static-domain

Conversation

@neefrehman

@neefrehman neefrehman commented Dec 16, 2022

Copy link
Copy Markdown
Contributor

Addresses recharts/recharts#3099, recharts/recharts#3103

Adds API documentation for the includeHidden axis prop introduced in recharts/recharts#3103. Having seen a contributor do the same in another PR, I've used Google Translate for the Chinese description. More than happy for this to be replaced by a better translation from a native speaker.

@ckifer

ckifer commented Dec 16, 2022

Copy link
Copy Markdown
Member

Don't merge until recharts/recharts#3103 is merged

ckifer pushed a commit to recharts/recharts that referenced this pull request Feb 24, 2023
…ta (#3103)

Closes #3099 

This PR introduces the `includeHidden` prop to `Axis` elements. Setting
this to true ensures that all datapoints within a chart contribute to
its domain calculation, even when they are hidden. This allows
developers to improve the end-user experience for charts that allow
toggling datapoints, as it can improve their ability to make comparisons
on narrower sets of data. Documentation for this feature is added in
recharts/recharts.org#223.

---------

Co-authored-by: Neef Rehman <[email protected]>
@neefrehman

Copy link
Copy Markdown
Contributor Author

@ckifer just bumping this PR now that the feature is merged

@ckifer

ckifer commented Feb 24, 2023

Copy link
Copy Markdown
Member

Will wait until next release to move it into the docs. Thanks!

@ckifer
ckifer merged commit b3a00c4 into recharts:master Mar 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants