Skip to content

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.

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