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

New Component for the Image and Text combination #26

Open
janine9vn opened this issue Aug 25, 2021 · 0 comments
Open

New Component for the Image and Text combination #26

janine9vn opened this issue Aug 25, 2021 · 0 comments

Comments

@janine9vn
Copy link
Contributor

The title isn't the most clear, but essentially I'm talking about this on the main page:
image

We use 3 of these on the same page and they all have nearly identical styling applied. It makes sense to reduce redundant code and instead create a React component out of it.

The React component should be able to accept an image, "title" text, description text, and then also specify the details for the CTAButton.

This may have to be split out into different React Components depending on if the image should go on the left or the right, but maybe not. I'm still a novice with React~

Implementation Considerations

I'd like to keep the styling of on hover over the image, the tint gets removed. This may not be able to work with next/image styling though, so I'm okay if it isn't technically feasible.

If we can keep it then I'd like to see if we can expand it so that if we hover over anywhere in the text then the tint on the image gets removes. That would be neat.

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

No branches or pull requests

1 participant