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

Added a details button that displays additional product information i… #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Chad-Martinez
Copy link

…n a modal. Upgraded to bootstrap 4.6 and added reactstrap. Touched up product card components with newer version of bootstrap. Made cards uniform height as suggested in issue #36. This PR was done as a part of a code challenge for a prospective employer. It is only a suggestion and does not need to be merged if the changes are undesired. Thank you for letting me contribute.

@krzysu
Copy link
Owner

krzysu commented Oct 27, 2021

hey, thanks! I'm not maintaining this project anymore, but your PR can be helpful to others, so I will just leave it 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.

None yet

2 participants