Skip to content

fix(ui): apply consistent my-8 to all markdown Alert usage#18235

Merged
pettinarip merged 3 commits into
devfrom
md-alert-spacing
May 26, 2026
Merged

fix(ui): apply consistent my-8 to all markdown Alert usage#18235
pettinarip merged 3 commits into
devfrom
md-alert-spacing

Conversation

@wackerow
Copy link
Copy Markdown
Member

@wackerow wackerow commented May 20, 2026

Description

  • Apply minimal 2rem vertical margins broadly to the Alert component for all markdown usage via MdComponents/index.tsx (additional margins from components above or below will automatically collapse and not add together)
  • Remove inconsistent use of custom margin overrides within markdown usage of Alert component
  • Leaves non-markdown usage to the consuming parent component -- unchanged
  • Patches external link styling, placing :not(.lucide-external-link) on svg selector to avoid re-coloring external link arrow icon within Alert component

Related Issue

design system

- Remove custom margin overrides within markdown usage of `Alert` component
- Apply minimal 2rem top and bottom margins broadly to the `Alert` component for all markdown usage via `MdComponents/index.tsx` (additional margins from components above or below will automatically collapse and not add together)
@wackerow wackerow requested a review from pettinarip as a code owner May 20, 2026 14:20
@wackerow wackerow added the design system this label will be used in all issues related to design system label May 20, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 20, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 6373803
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/6a117da1e618890008a35711
😎 Deploy Preview https://deploy-preview-18235.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 70 (🟢 up 5 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 98 (🔴 down 1 from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the content 🖋️ This involves copy additions or edits label May 20, 2026
myelinated-wackerow and others added 2 commits May 23, 2026 03:10
- Updates [svg] to [svg:not(.lucide-external-link)] for SVG coloring in alert variants
- Avoids re-coloring the external link arrow within Alert component

Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
@pettinarip pettinarip merged commit ed83c4c into dev May 26, 2026
15 checks passed
@pettinarip pettinarip deleted the md-alert-spacing branch May 26, 2026 17:16
@pettinarip pettinarip mentioned this pull request May 28, 2026
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 design system this label will be used in all issues related to design system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants