Skip to content

Conversation

@GomezIvann
Copy link
Collaborator

@GomezIvann GomezIvann commented May 23, 2024

Checklist

  • The build process is done without errors. All tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Description
New DxcBarChart component added to the Halstack Design System. Based on the Figma designs below, it uses Cloudscape's component under the hood.

image

Props table
https://nettle-opera-f35.notion.site/Bar-Chart-d8092551790b467f8e611f0f8033ee91?pvs=4

Additional changes
Some accessibility issues were reported at the Spinner component when the component was undetermined and the mode was small. Both aria-label and aria-labelledby attributes have been included to solve this issues.

image

Additional context
Cloudscape Bar Chart
Halstack Bar Chart Designs

@GomezIvann GomezIvann marked this pull request as ready for review May 23, 2024 11:00
@GomezIvann GomezIvann changed the title New Bar Chart component New DxcBarChart component May 23, 2024
@Jialecl Jialecl self-requested a review May 24, 2024 10:03
@Jialecl Jialecl self-assigned this May 24, 2024
Jialecl
Jialecl previously approved these changes May 30, 2024
@Jialecl Jialecl merged commit c667373 into master May 30, 2024
@Jialecl Jialecl deleted the gomezivann/bar-chart branch May 30, 2024 12:00
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