Skip to content

Display tags in commit table UI #24283

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Display tags in commit table UI #24283

wants to merge 2 commits into from

Conversation

lunny
Copy link
Member

@lunny lunny commented Apr 23, 2023

Fix #20167

@lunny lunny added the type/enhancement An improvement of existing functionality label Apr 23, 2023
@GiteaBot GiteaBot added this to the 1.20.0 milestone Apr 23, 2023
@lunny lunny mentioned this pull request Apr 23, 2023
Closed
@silverwind silverwind added type/feature Completely new functionality. Can only be merged if feature freeze is not active. and removed type/enhancement An improvement of existing functionality labels Apr 23, 2023
@silverwind
Copy link
Member

Think this classifies as a feature.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 23, 2023
@silverwind
Copy link
Member

silverwind commented Apr 23, 2023 •
edited
Loading

We may want to refactor the commits table before this to no longer be a table, but a div list with flexbox, to allow much better styling via CSS that tables can not do.

@silverwind
Copy link
Member

Watch #24423.

@lunny lunny modified the milestones: 1.20.0, 1.21.0 May 10, 2023
@delvh delvh removed this from the 1.21.0 milestone May 10, 2023
@lunny
Copy link
Member Author

lunny commented Jul 2, 2023

should be replaced by #24755

@lunny lunny closed this Jul 2, 2023
@lunny lunny deleted the lunny/display_tags_commits branch July 2, 2023 07:59
@silverwind
Copy link
Member

I hope I can get back to commit list rework soon, then we can talk about this feature.

@go-gitea go-gitea locked as resolved and limited conversation to collaborators Sep 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Highlight/mark tags and branches in commit view
4 participants
@lunny @silverwind @GiteaBot @delvh