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

Full y axis labels in Heatmap chart #33509

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Anush2303
Copy link
Contributor

New Behavior

Full Y Axis labels feature in Heatmap chart.

image

Related Issue(s)

  • Fixes #

@Anush2303 Anush2303 marked this pull request as ready for review December 23, 2024 06:31
@Anush2303 Anush2303 requested a review from a team as a code owner December 23, 2024 06:31
Copy link

github-actions bot commented Dec 23, 2024

📊 Bundle size report

✅ No changes found

@@ -0,0 +1,7 @@
{
Copy link
Collaborator

@fabricteam fabricteam Dec 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🕵🏾‍♀️ visual regressions to review in the fluentuiv8 Visual Regression Report

Callout 6 screenshots
Image Name Diff(in Pixels) Image Type
Callout.Left center.chromium.png 2616 Changed
Callout.Top center.chromium.png 2178 Changed
Callout.Top left edge.chromium.png 2307 Changed
Callout.No callout width specified.chromium.png 2319 Changed
Callout.Bottom auto edge.chromium.png 2309 Changed
Callout.No beak.chromium.png 2306 Changed
Keytip 1 screenshots
Image Name Diff(in Pixels) Image Type
Keytip.Offset.chromium.png 121 Changed
react-charting-HeatMapChart 2 screenshots
Image Name Diff(in Pixels) Image Type
react-charting-HeatMapChart.Basic.chromium.png 307 Changed
react-charting-HeatMapChart.Basic Dark Mode.chromium.png 320 Changed
react-charting-LineChart 2 screenshots
Image Name Diff(in Pixels) Image Type
react-charting-LineChart.Events.chromium.png 2 Changed
react-charting-LineChart.Gaps.chromium.png 1 Changed
react-charting-VerticalBarChart 1 screenshots
Image Name Diff(in Pixels) Image Type
react-charting-VerticalBarChart.Basic - Secondary Y Axis.chromium.png 4 Changed

Copy link

Pull request demo site: URL

/**
*@default false
*Used for showing complete y axis lables */
showYAxisLables?: boolean;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

showYAxisLables

his prop is already part of cartesian. Should not be needed here.

Copy link
Contributor Author

@Anush2303 Anush2303 Dec 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build is failing without this. Because, in plotlyadapter file, we don't have reference to ICartesianchartProps. This prop is defined in HBCWithAxis also, so I think it is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants