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

Refactor HorizontalCard for Chakra UI #7893

Merged
merged 7 commits into from
Nov 4, 2022
Merged

Conversation

ameeetgaikwad
Copy link
Contributor

Description

Refactored HorizontalCard component for Chakra UI.

@gatsby-cloud
Copy link

gatsby-cloud bot commented Sep 17, 2022

✅ ethereum-org-website-dev deploy preview ready

Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

Good job @amm98d thanks for opening a new PR.

src/components/HorizontalCard.tsx Outdated Show resolved Hide resolved
src/components/HorizontalCard.tsx Outdated Show resolved Hide resolved
src/components/HorizontalCard.tsx Outdated Show resolved Hide resolved
src/components/HorizontalCard.tsx Outdated Show resolved Hide resolved
src/components/HorizontalCard.tsx Outdated Show resolved Hide resolved
Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

For some reason, this version doesn't look the same as in prod. Check the following examples. They are from https://ethereum.org/en/stablecoins/

this PR version:
image

prod:
image

I encourage you to try to see where the issue is. If not, later I'll try to debug it myself.

@ameeetgaikwad
Copy link
Contributor Author

@pettinarip not able to find it :(

@pettinarip pettinarip self-assigned this Sep 26, 2022
Copy link
Contributor

@TylerAPfledderer TylerAPfledderer left a comment

Choose a reason for hiding this comment

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

Hello! 😄
Left a comment regarding the outstanding issue.

<Description>{description}</Description>
<Flex borderRadius="base" className={className}>
<Emoji fontSize={`${emojiSize}rem`} text={emoji} />
<Flex grow="0" shrink="1" basis="75%" ml="8">
Copy link
Contributor

Choose a reason for hiding this comment

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

@ameeetgaikwad and @pettinarip: Your culprit is this Flex component, when it should just be a Box. You will need to reword the other props.

<Box flexGrow='0' flexShrink='1' flexBasis='75%' ml="8">

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll work on that

Copy link
Member

Choose a reason for hiding this comment

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

Thanks, @ameeetgaikwad and @TylerAPfledderer. Will circle back in a couple of days

Copy link
Contributor Author

@ameeetgaikwad ameeetgaikwad Oct 25, 2022

Choose a reason for hiding this comment

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

@pettinarip @corwintines check this out! I've done the required changes

Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

lgtm! thanks @ameeetgaikwad

@pettinarip pettinarip merged commit 9f4c97a into ethereum:dev Nov 4, 2022
@gitpoap-bot
Copy link

gitpoap-bot bot commented Nov 4, 2022

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2022 Ethereum.org Contributor:

GitPOAP: 2022 Ethereum.org Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

@pettinarip
Copy link
Member

@ameeetgaikwad be sure to join the discord if you are interested in contributing further to the project or have any questions for the team. And we've just released our 2022 POAPs so remember to claim yours also 🥳!

@pettinarip
Copy link
Member

@all-contributors please add @ameeetgaikwad for code

@allcontributors
Copy link
Contributor

@pettinarip

I've put up a pull request to add @ameeetgaikwad! 🎉

@allcontributors
Copy link
Contributor

@ameeetgaikwad

I've put up a pull request to add @jakebolam! 🎉

@allcontributors
Copy link
Contributor

@ameeetgaikwad

I've put up a pull request to add @ameeetgaikwad! 🎉

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

Successfully merging this pull request may close these issues.

4 participants