Skip to content

Comments

[6.x] Increase inline padding of badges#14002

Merged
jasonvarga merged 1 commit into6.xfrom
tweak-badge-x-padding
Feb 20, 2026
Merged

[6.x] Increase inline padding of badges#14002
jasonvarga merged 1 commit into6.xfrom
tweak-badge-x-padding

Conversation

@jaygeorge
Copy link
Contributor

@jaygeorge jaygeorge commented Feb 19, 2026

Description of the Problem

  • The space between the edges of the text and the radii of the badges is a little too tight, especially when caps are used e.g. 100 KB
  • Increasing the inline padding slightly makes it feel better to my eyes

What this PR Does

  • Increases the inline padding of the standard badge size so that text fits more comfortably

Before

2026-02-19 at 13 06 56@2x

After

2026-02-19 at 13 07 54@2x

How to Reproduce

  1. You can see this it the asset meta badges if you open up an asset in the cp

@daun
Copy link
Contributor

daun commented Feb 19, 2026

Which image format is that? Interesting that it allows subpixel image dimensions (114.38815px height).

@jaygeorge
Copy link
Contributor Author

Oh hah, I didn't even notice that. Probably a separate issue—it wasn't me, your honor!

@daun
Copy link
Contributor

daun commented Feb 19, 2026

Is it an SVG? Just remembered I've seen subpixel SVGs in use sometimes. No issue, then, but might still make sense to round to 1 decimal place in display.

@jaygeorge
Copy link
Contributor Author

It was an SVG indeed! I'll submit a separate PR, cheers Daun 🤘

@jasonvarga
Copy link
Member

Me trying to see the padding:

senor-chang-gif

(jk I see it) Your attention to detail is 👌✨

@jasonvarga jasonvarga changed the title [6.x] Increase inline padding of badge slightly based on asset meta badges feeli… [6.x] Increase inline padding of badges Feb 20, 2026
@jasonvarga jasonvarga merged commit b1c12ee into 6.x Feb 20, 2026
14 checks passed
@jasonvarga jasonvarga deleted the tweak-badge-x-padding branch February 20, 2026 21:13
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