Skip to content

Comments

Remove padding override on .ui .sha.label (#35864)#35873

Merged
silverwind merged 2 commits intogo-gitea:release/v1.25from
GiteaBot:backport-35864-v1.25
Nov 6, 2025
Merged

Remove padding override on .ui .sha.label (#35864)#35873
silverwind merged 2 commits intogo-gitea:release/v1.25from
GiteaBot:backport-35864-v1.25

Conversation

@GiteaBot
Copy link
Collaborator

@GiteaBot GiteaBot commented Nov 6, 2025

Backport #35864 by @silverwind

Since upgrading to v1.25, I noticed the SHA labels have slightly different padding than before. I can't pinpoint exactly which change it was. Fix it by removing the padding override on .ui .sha.label and make the one on.ui.label (2px 6px) take effect which matches 1.24 rendering.

Before:

image

After:

image

Since upgrading to v1.25, I noticed the SHA labels have slightly
different padding than before. I can't pinpoint exactly which change it
was. Fix it by removing the padding override on `.ui .sha.label` and
make the one on`.ui.label` (`2px 6px`) take effect which matches 1.24
rendering.

Before:

<img width="135" height="172" alt="image"
src="https://github.com/user-attachments/assets/2781a854-be08-4a11-bde0-d3699b2b7454"
/>

After:

<img width="139" height="162" alt="image"
src="https://github.com/user-attachments/assets/5c864fa3-c1f9-4452-ae58-5411dd445865"
/>
@GiteaBot GiteaBot added this to the 1.25.2 milestone Nov 6, 2025
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 6, 2025
@GiteaBot GiteaBot requested review from delvh and wxiaoguang November 6, 2025 01:32
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 6, 2025
@silverwind silverwind enabled auto-merge (squash) November 6, 2025 01:45
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Nov 6, 2025
@silverwind silverwind added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 6, 2025
@silverwind silverwind merged commit 289bd96 into go-gitea:release/v1.25 Nov 6, 2025
26 checks passed
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 6, 2025
@xnox xnox mentioned this pull request Dec 7, 2025
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Feb 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants