Skip to content

fix: tighten code display for ascii art#363

Merged
danielroe merged 1 commit intonpmx-dev:mainfrom
posva:fix/css-readme-ascii
Jan 29, 2026
Merged

fix: tighten code display for ascii art#363
danielroe merged 1 commit intonpmx-dev:mainfrom
posva:fix/css-readme-ascii

Conversation

@posva
Copy link
Copy Markdown
Contributor

@posva posva commented Jan 29, 2026

I noticed that some ascii art wasn't dispalying correctly. I didn't test this throughout but it seemed like a quick fix.

Before the fix:

CleanShot 2026-01-29 at 22 42 07@2x

After the fix:

CleanShot 2026-01-29 at 22 42 14@2x

red lines are for debug

Checked on https://npmx.dev/table

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 29, 2026

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

Project Deployment Actions Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment Jan 29, 2026 9:50pm
npmx.dev Ready Ready Preview, Comment Jan 29, 2026 9:50pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Jan 29, 2026 9:50pm

Request Review

@danielroe danielroe added this pull request to the merge queue Jan 29, 2026
Merged via the queue into npmx-dev:main with commit 21b466a Jan 29, 2026
13 checks passed
@okineadev
Copy link
Copy Markdown
Contributor

I see this on my phone right now:

Screenshot_20260130_092536_Chrome.png Screenshot_20260130_092601_Chrome.png Screenshot_20260130_092605_Chrome.png

@posva posva deleted the fix/css-readme-ascii branch January 30, 2026 08:26
@posva
Copy link
Copy Markdown
Contributor Author

posva commented Jan 30, 2026

I can't repro on ios, they do appear slightly off on both pc and phone
CleanShot 2026-01-30 at 09 29 00

I think this is because of the mono font itself, maybe something about fallbacks on your device?

Using monospace, it looks better:

CleanShot 2026-01-30 at 09 30 58

@okineadev
Copy link
Copy Markdown
Contributor

I can't repro on ios, they do appear slightly off on both pc and phone
CleanShot 2026-01-30 at 09 29 00

I think this is because of the mono font itself, maybe something about fallbacks on your device?

Using monospace, it looks better:

CleanShot 2026-01-30 at 09 30 58

So there may be a problem with the fonts

We have to look into this

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.

3 participants