Skip to content

Conversation

@joneskj55
Copy link
Contributor

Description

This PR renders the StatsBoxGrid result as a span as opposed to an h3 element.

Related Issue

#4757

With change

Screen Shot 2021-12-14 at 12 48 06 PM

Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

@joneskj55 Thanks! LGTM =)

Copy link
Member

@corwintines corwintines left a comment

Choose a reason for hiding this comment

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

Thanks @joneskj55!

Copy link
Contributor

@minimalsm minimalsm left a comment

Choose a reason for hiding this comment

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

Thanks for this @joneskj55.

Made a small tweak from px to rem in the name of accessibility while I was here :-)

@minimalsm minimalsm merged commit 5c40644 into ethereum:dev Dec 15, 2021
@joneskj55 joneskj55 deleted the stats-box-grid-span branch December 15, 2021 13:57
@joneskj55
Copy link
Contributor Author

@minimalsm Cool, and thanks for the a11y link!

@wackerow wackerow mentioned this pull request Dec 16, 2021
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.

4 participants