Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Display head branch more comfortable on pull request view #32000

Merged
merged 4 commits into from
Sep 24, 2024
Merged

Display head branch more comfortable on pull request view #32000

merged 4 commits into from
Sep 24, 2024

Conversation

lunny
Copy link
Member

@lunny lunny commented Sep 6, 2024

This PR do some minor improvements for head branch display on pull request view UI.

  • Remove the link if the head branch has been deleted with a tooltip, so that users will not result in a 404 page
  • Display a label if this pull request is an agit based one.
    图片

@lunny lunny added the topic/ui Change the appearance of the Gitea UI label Sep 6, 2024
@lunny lunny added this to the 1.23.0 milestone Sep 6, 2024
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 6, 2024
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 6, 2024
@github-actions github-actions bot added modifies/translation modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files labels Sep 6, 2024
@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 Sep 12, 2024
@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 Sep 24, 2024
@yp05327 yp05327 added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Sep 24, 2024
@lunny lunny enabled auto-merge (squash) September 24, 2024 02:46
@lunny lunny merged commit e2f365b into go-gitea:main Sep 24, 2024
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 Sep 24, 2024
zjjhot added a commit to zjjhot/gitea that referenced this pull request Sep 25, 2024
@zjjhot
Merge remote-tracking branch 'giteaofficial/main'
8b9f98b 
* giteaofficial/main:
  Notify the user when the file path contains leading or trailing spaces and fix the error message for invalid file names. (go-gitea#31507)
  Fix wrong status of `Set up Job` when first step is skipped (go-gitea#32120)
  Fix bug when deleting a migrated branch (go-gitea#32075)
  Include collaboration repositories on dashboard source/forks/mirrors list (go-gitea#31946)
  Display head branch more comfortable on pull request view (go-gitea#32000)
  Truncate commit message during Discord webhook push events (go-gitea#31970)
  Fix template bug of pull request view (go-gitea#32072)
@Zettat123 Zettat123 mentioned this pull request Oct 9, 2024
Merged
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Dec 23, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers

@wolfogre wolfogre wolfogre approved these changes

@yp05327 yp05327 yp05327 approved these changes

Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/translation size/S Denotes a PR that changes 10-29 lines, ignoring generated files. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Milestone
1.23.0
Development

Successfully merging this pull request may close these issues.
4 participants
@lunny @wolfogre @yp05327 @GiteaBot