Skip to content

Tag tweaks #13426

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

Closed
2 of 6 tasks
jolheiser opened this issue Nov 4, 2020 · 6 comments · Fixed by #13428
Closed
2 of 6 tasks

Tag tweaks #13426

jolheiser opened this issue Nov 4, 2020 · 6 comments · Fixed by #13428
Labels
topic/ui Change the appearance of the Gitea UI

Comments

@jolheiser
Copy link
Member

jolheiser commented Nov 4, 2020

  • Gitea version (or commit ref): 1.14.0+dev-148-gfb756e773
  • Git version: Whatever try uses
  • Operating system: Whatever try uses
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite (does try still use SQLite, or did we change it?)
  • Can you reproduce the bug at https://try.gitea.io:
  • Log gist:
    N/A

Description

As noted in the above link, the repo has 89 tags, but none are true Gitea releases.

Previously this was inconsequential because we showed both (and we still treat tags as releases in the DB, albeit with IsTag: true)

However, now that we distinguish tags on a separate page and added the number to the sub menu, we should specifically calculate how many true releases we have vs how many are simple tags, and show them in the corresponding spot on the UI.

imo the sub menu (next to branches) should show the number of all tags (because releases are tags, but tags aren't true releases)
The top menu should only show the number of real releases.

The sub menu should also be renamed "tags"

Screenshots

kawa

@jolheiser jolheiser added the topic/ui Change the appearance of the Gitea UI label Nov 4, 2020
@silverwind
Copy link
Member

silverwind commented Nov 4, 2020

Yes, we need separate counters for both releases and tags. I guess to not add a 5th duplicating entry in the second list, it's fine to just rename that to Tags.

@jolheiser
Copy link
Member Author

Yes, we need separate counters for both releases and tags. I guess to not add a 5th duplicating entry in the second list, it's fine to just rename that to Tags.

My reasoning is that the second (sub) menu relates to git data, in which case tags makes sense because git has no concept of a formal release.

Top menu calculation is the current NumReleases, but the second (sub) menu calculation is only releases where IsTag: true

@silverwind
Copy link
Member

silverwind commented Nov 4, 2020

My thoughts. Git does have annotated tags but I guess only 1% of users would know the difference between a lightweight and an annotated tag so I guess we follow what GitHub does here.

I do personally use annotated tags for all my releases (primarly because I can't be bothered with doing it on the GUI), maybe they can be optionally be promoted to be seen as a proper release by Gitea later e.g. COUNT_ANNOTATED_TAG_AS_RELEASE or something.

@jolheiser
Copy link
Member Author

Even with annotated tags, git doesn't track attachments, pre-release, etc.

(But I do agree having a Gitea release created by pushing an annotated tag would be 👌 )

Once Gitea can distinguish between lightweight and annotated we could certainly revisit the counts.

@silverwind
Copy link
Member

One slightly related topic is also to display tags on the release list in a deemphasized form like GitHub does, I think it's a nice feature.

image

@jolheiser
Copy link
Member Author

One slightly related topic is also to display tags on the release list in a deemphasized form like GitHub does, I think it's a nice feature.

I chose not to implement this in the linked PR, but I do agree it's nice to have. Might be worth adding in another PR.

@go-gitea go-gitea locked and limited conversation to collaborators Dec 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants