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

Remove italic tags from SocialListItem markdown #7037

Merged
merged 1 commit into from
Jul 13, 2022

Conversation

minimalsm
Copy link
Contributor

Description

Our online communities page was using <i> tags inside of a React component. But, Crowdin rebuilds any HTML tag in markdown using markdown syntax (underscores instead of <i> tags) causing the styling to not be applied as markdown syntax doesn't work inside a React component. This PR moves that styling into a Styled children container.

Error in markdown:
Screenshot 2022-07-13 at 12 15 15

Error on page:
Screenshot 2022-07-13 at 12 49 49

Related Issue

None

@github-actions github-actions bot added content 🖋️ This involves copy additions or edits review needed 👀 labels Jul 13, 2022
@gatsby-cloud
Copy link

gatsby-cloud bot commented Jul 13, 2022

Gatsby Cloud Build Report

ethereum-org-website-dev

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 35m

Performance

Lighthouse report

Metric Score
Performance 🔶 19
Accessibility 🔶 89
Best Practices 💚 100
SEO 💚 92

🔗 View full report

@wackerow wackerow merged commit 791d029 into dev Jul 13, 2022
@wackerow wackerow deleted the patchItalicsSocialListItem branch July 13, 2022 18:03
@wackerow wackerow mentioned this pull request Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants