-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
Fix issue close timeline icon #36138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think issue closed is a terrible octicon as circle slash is much more descriptive. But oh well…
|
I guess the intend of that icon is to evoke more positivity 😆. |
|
Maybe it depends on why it will be closed which is an unimplemented feature. |
|
Yes, I think we definitely need to implement close reasons. |
* giteaofficial/main: Remove undocumented support of signing key in the repository git configuration file (go-gitea#36143) Enable gocheckcompilerdirectives linter (go-gitea#36156) Fix code highlighting on blame page (go-gitea#36157) Check user visibility when redirecting to a renamed user (go-gitea#36148) Fix bug when viewing the commit diff page with non-ANSI files (go-gitea#36149) Refactor `FileTreeItem` type (go-gitea#36137) Fix various bugs (go-gitea#36139) Fix issue close timeline icon (go-gitea#36138) Add permissions to`files-changed` jobs (go-gitea#36142) Add explicit permissions to all actions workflows (go-gitea#36140) Bump `actions/checkout` to v6 (go-gitea#36136) Hide RSS icon when viewing a file not under a branch (go-gitea#36135) Fix SVG size calulation, only use `style` attribute (go-gitea#36133) Add sorting/filtering to admin user search API endpoint (go-gitea#36112)
Previously there was a icon mismatch between a issue's label and the timeline close event icon:
This fixes it and the icons match now: