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

feat: allow multiple DynamicTable per page MAASENG-3091 #129

Conversation

petermakowski
Copy link
Contributor

@petermakowski petermakowski commented Apr 24, 2024

Done

  • feat: allow multiple DynamicTable per page MAASENG-3091
  • add variant prop
  • use React's Context API for managing the variant prop.

Note: Variant prop is required deliberately so that we're careful when updating existing implementations and carefully chose which variant should be used.

Why Context API?

Context API provides a way to pass data through the component tree without having to pass props down manually at every level. This also allows to use this prop in all child components and make sure it's consistent and cannot be overwritten.

QA steps

  • Ensure the component is displayed correctly across all breakpoints
  • Verify it works correctly in supported browsers (at least Firefox and Chrome)
  • Make sure headers and body cells are aligned in both variants.

Fixes

Fixes: https://warthogs.atlassian.net/browse/MAASENG-3091 #128

Screenshots

After

Google Chrome screenshot 001853@2x

Notes

@webteam-app
Copy link
Collaborator

@petermakowski petermakowski force-pushed the feat-allow-multiple-DynamicTable-components-per-page-MAASENG-3091 branch from cb90c75 to d8b4bda Compare April 24, 2024 13:59
@petermakowski petermakowski force-pushed the feat-allow-multiple-DynamicTable-components-per-page-MAASENG-3091 branch 2 times, most recently from 49095cf to e0c4462 Compare April 24, 2024 14:11
@petermakowski petermakowski marked this pull request as ready for review April 24, 2024 14:11
@petermakowski petermakowski force-pushed the feat-allow-multiple-DynamicTable-components-per-page-MAASENG-3091 branch from e0c4462 to 41d5d88 Compare April 24, 2024 14:17
@petermakowski petermakowski force-pushed the feat-allow-multiple-DynamicTable-components-per-page-MAASENG-3091 branch from 41d5d88 to 2d41e0b Compare April 24, 2024 14:22
Copy link
Collaborator

@ndv99 ndv99 left a comment

Choose a reason for hiding this comment

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

Sorry this took a while, wanted to do some testing and hacking around to make sure this works as expected, but I think this is good to merge.

@petermakowski petermakowski merged commit 0631b2a into canonical:main Apr 25, 2024
7 checks passed
Copy link

🎉 This PR is included in version 1.28.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants