Lots of unnecessary/repetitive unlocked icons in commit list #14043

Closed
opened 2025-11-02 11:01:07 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @ghost on GitHub (Jan 25, 2025).

Description

Too many unlocked icon in commit list, especially for unsigned commits, simply no need of an "unverified" icon, unless adding vigilant mode as what GitHub does. (example on demo site: https://demo.gitea.com/lunny/TheSession-data/commits/branch/main)

https://github.com/go-gitea/gitea/pull/10425 looks quite smooth and distinguished.
While changes from https://github.com/go-gitea/gitea/pull/31160 simply determines the icon display by whether commit signed (verified) or not. 079a1ffe8f/templates/repo/commit_sign_badge.tmpl

Screenshots

Image

Image

Gitea Version

7da8a01d

Can you reproduce the bug on the Gitea demo site?

Yes

Operating System

No response

Browser Version

Firefox 134.0

Originally created by @ghost on GitHub (Jan 25, 2025). ### Description Too many unlocked icon in commit list, especially for unsigned commits, simply no need of an "unverified" icon, unless adding vigilant mode as what GitHub does. (example on demo site: https://demo.gitea.com/lunny/TheSession-data/commits/branch/main) https://github.com/go-gitea/gitea/pull/10425 looks quite smooth and distinguished. While changes from https://github.com/go-gitea/gitea/pull/31160 simply determines the icon display by whether commit signed (verified) or not. https://github.com/go-gitea/gitea/blob/079a1ffe8f38e0c7570079b21a4d8cec4d90e923/templates/repo/commit_sign_badge.tmpl ### Screenshots ![Image](https://github.com/user-attachments/assets/33a9cfb4-3fa3-4282-a72a-5e3f34f0e284) ![Image](https://github.com/user-attachments/assets/b1c38651-a5da-4cc8-ac00-96d5fea122c0) ### Gitea Version 7da8a01d ### Can you reproduce the bug on the Gitea demo site? Yes ### Operating System _No response_ ### Browser Version Firefox 134.0
GiteaMirror added the type/bug label 2025-11-02 11:01:07 -06:00
Author
Owner

@lunny commented on GitHub (Jan 25, 2025):

I think this proposal is acceptable from this side but that's not a bug.

@lunny commented on GitHub (Jan 25, 2025): I think this proposal is acceptable from this side but that's not a bug.
Author
Owner

@wxiaoguang commented on GitHub (Jan 25, 2025):

I think it is a bug, we should avoid showing unnecessary locks.

@wxiaoguang commented on GitHub (Jan 25, 2025): I think it is a bug, we should avoid showing unnecessary locks.
Author
Owner

@wxiaoguang commented on GitHub (Jan 25, 2025):

The fix could be like this: Do not show unnecessary commit sign lock icon #33387

@wxiaoguang commented on GitHub (Jan 25, 2025): The fix could be like this: Do not show unnecessary commit sign lock icon #33387
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#14043