Skip to content

Comments

SUR-305 - Added Y Axis tick formatter#248

Merged
vrundakansara merged 3 commits intodevfrom
bar_chart_fix
Jan 21, 2025
Merged

SUR-305 - Added Y Axis tick formatter#248
vrundakansara merged 3 commits intodevfrom
bar_chart_fix

Conversation

@ravindrakele
Copy link
Contributor

@ravindrakele ravindrakele commented Jan 9, 2025

Description

https://d.pr/v/triQSY

You can use Y axis tick formatter like below

<BarChart
     yTickFormatter={
           ( value ) => value ? value : 'n/a'     }
/>

Screenshots

Types of changes

How has this been tested?

Checklist:

  • My code is tested
  • My code passes the PHPCS tests
  • I've created the npm build.
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

@ravindrakele ravindrakele changed the title Added Y Axis tick formatter SUR-305 - Added Y Axis tick formatter Jan 13, 2025
Copy link
Contributor

@jaieds jaieds left a comment

Choose a reason for hiding this comment

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

Can you add a story for it?

@ravindrakele ravindrakele requested a review from jaieds January 21, 2025 06:39
@ravindrakele
Copy link
Contributor Author

Can you add a story for it?

Added the story
https://d.pr/i/dihe8R

@ravindrakele ravindrakele requested a review from jaieds January 21, 2025 07:12
@vrundakansara vrundakansara merged commit beb657a into dev Jan 21, 2025
4 checks passed
@vrundakansara vrundakansara deleted the bar_chart_fix branch January 21, 2025 09:12
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.

3 participants