Skip to content

fix(jsx-email): show line numbers in code views#451

Merged
shellscape merged 1 commit into
mainfrom
fix/jsx-email/preview-line-nos
May 17, 2026
Merged

fix(jsx-email): show line numbers in code views#451
shellscape merged 1 commit into
mainfrom
fix/jsx-email/preview-line-nos

Conversation

@shellscape
Copy link
Copy Markdown
Owner

@shellscape shellscape commented May 17, 2026

Component / Package Name:

jsx-email

This PR contains:

  • bugfix
  • feature
  • refactor
  • documentation
  • other

Are tests included?

  • yes (bugfixes and features will not be merged without tests)
  • no

Breaking Changes?

  • yes (breaking changes will not be merged unless absolutely necessary)
  • no

If yes, please include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.

List any relevant issue numbers:

N/A

Description

Adds Shiki-backed line numbers to the shipped preview app code views for JSX, HTML, and plain text content. The gutter spacing and line content alignment follow the existing samples/docs code block style so source views remain scannable while preserving copy/download behavior.

Preview line numbers

Verification

  • ./node_modules/.bin/moon run repo:build.all --cache off
  • ./node_modules/.bin/moon run :lint
  • ./node_modules/.bin/moon run :typecheck
  • ./node_modules/.bin/moon run :test
  • ./node_modules/.bin/moon run smoke-v2:run

@shellscape shellscape force-pushed the fix/jsx-email/preview-line-nos branch from 4d691e8 to 22dc65d Compare May 17, 2026 02:28
@shellscape shellscape merged commit 7a175de into main May 17, 2026
8 of 9 checks passed
@shellscape shellscape deleted the fix/jsx-email/preview-line-nos branch May 17, 2026 02:45
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.

1 participant