Skip to content

Conversation

alexcambose
Copy link
Contributor

Summary

  • Mdx is wrapping every text component into a <p> which was causing extra spacing (https://github.com/orgs/mdx-js/discussions/2194)
  • Wrapped the contexts of the Info box into <div>s which is not great but fixes the issue
  • Added storybook section for the <InfoBox/>

Rationale

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@alexcambose alexcambose requested a review from a team as a code owner September 12, 2025 16:50
Copy link

vercel bot commented Sep 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
component-library Ready Ready Preview Comment Sep 12, 2025 5:29pm
developer-hub Ready Ready Preview Comment Sep 12, 2025 5:29pm
5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Sep 12, 2025 5:29pm
entropy-explorer Skipped Skipped Sep 12, 2025 5:29pm
insights Skipped Skipped Sep 12, 2025 5:29pm
proposals Skipped Skipped Sep 12, 2025 5:29pm
staking Skipped Skipped Sep 12, 2025 5:29pm

@alexcambose alexcambose changed the title fix: info boxes fix(dev-hub): info boxes Sep 12, 2025
@vercel vercel bot temporarily deployed to Preview – proposals September 12, 2025 17:26 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer September 12, 2025 17:26 Inactive
@vercel vercel bot temporarily deployed to Preview – staking September 12, 2025 17:26 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference September 12, 2025 17:26 Inactive
@vercel vercel bot temporarily deployed to Preview – insights September 12, 2025 17:26 Inactive
@alexcambose alexcambose merged commit fa234c7 into main Sep 12, 2025
10 checks passed
@alexcambose alexcambose deleted the feat/dev-hub branch September 12, 2025 17:37
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.

2 participants