search

jakearchibald / svgomg

Web GUI for SVGO
https://jakearchibald.github.io/svgomg/
MIT License
5.76k stars 476 forks source link

Improve Error messages display #353

Closed XhmikosR closed 2 years ago

XhmikosR commented 2 years ago
Main First patch Second patch
1 2 3

The button's height should probably be limited. Feel free to push any improvements and let me know which approach you prefer.

netlify[bot] commented 2 years ago

✔️ Deploy Preview for svgomg ready!

🔨 Explore the source changes: 9ef1ab0589ffa54bafce0d4f967aa6139ea13359

🔍 Inspect the deploy log: https://app.netlify.com/sites/svgomg/deploys/6183d21f23c33700092a91e3

😎 Browse the preview: https://deploy-preview-353--svgomg.netlify.app/

XhmikosR commented 2 years ago

So you prefer the second patch, right?

jakearchibald commented 2 years ago

So you prefer the second patch, right?

Yes

jakearchibald commented 2 years ago

Before I do a full review, is the "second patch" the one represented by this PR?

XhmikosR commented 2 years ago

Yup. It's the one with the monospaced font.