Component | Area: Different min height implementation #550
Conversation
- Introduced `minHeight` property for minimum area height, defaulting to undefined. - Updated area generation logic to accommodate new `minHeight` configuration. - Deprecated `minHeight1Px` in favor of `minHeight` for better clarity and functionality.
| If your data has small or zero values leading to some parts of the area to become invisible, you can force those area segments to have 1px height | ||
| despite their actual value by setting `minHeight1Px` to `true`. This can be useful if you want to visually emphasize that the data behind the chart | ||
| is defined but just very small. | ||
| If your data has small or zero values leading to some parts of the area to become invisible, you can use the `minHeight` property to set a minimum height for the area. This can be useful if you want to visually emphasize that the data behind the chart is defined but just very small. |
There was a problem hiding this comment.
Do we want to add a note there indicating we want to deprecate minHeight1px? Otherwise right now there's no mention on the website, just 2 examples right next to each other
There was a problem hiding this comment.
@lee00678 I was thinking about it but decided not to (forgot to remove the example). The property will be marked as deprecated in the code asking users to use minHeight
There was a problem hiding this comment.
@lee00678 I was thinking about it but decided not to (forgot to remove the example). The property will be marked as deprecated in the code asking users to use minHeight
There was a problem hiding this comment.
@rokotyan I will remove the minHeight1px example then
|
@rokotyan this looks good to me. |
- Revised the explanation of the `minHeight` property to clarify its purpose and usage. - Added a note regarding the visual representation of data when using `minHeight` to ensure users understand its impact on the chart display.
rokotyan
force-pushed
the
feature/area-min-height
branch
from
9689d53 to
fc96812
Compare
New
minHeightconfig, deprecatingminHeight1Px.minHeightproperty for minimum area height, defaulting to undefined.minHeightconfiguration.minHeight1Pxin favor ofminHeightfor better clarity and functionality.Dev Example
Updated Docs: