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 6 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 7 screenshots
Image Name Diff(in Pixels) Image Type
Callout.Right bottom edge.chromium.png 3005 Changed
Callout.Bottom auto edge.chromium.png 2309 Changed
Callout.Rendering callout attached to a rectangle.chromium.png 1493 Changed
Callout.Left top edge.chromium.png 1949 Changed
Callout.No beak.chromium.png 2306 Changed
Callout.Beak 25.chromium.png 2186 Changed
Callout.Left bottom edge.chromium.png 3022 Changed
react-charting-HeatMapChart 2 screenshots
Image Name Diff(in Pixels) Image Type
react-charting-HeatMapChart.Basic Dark Mode.chromium.png 320 Changed
react-charting-HeatMapChart.Basic.chromium.png 307 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

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.

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