diff options
| author | yp05327 <576951401@qq.com> | 2024-01-22 00:13:00 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-01-21 15:13:00 +0000 |
| commit | b693611b35c5ae17cfc820bc3e731608a5251464 (patch) | |
| tree | 43c7529bb7f3778fb01482ed8d57a08a4952ee95 /templates/repo | |
| parent | 5d09023f1354f7a62f2ff68731e62e3e3645f4f2 (diff) | |
| download | gitea-b693611b35c5ae17cfc820bc3e731608a5251464.tar.gz gitea-b693611b35c5ae17cfc820bc3e731608a5251464.zip | |
Don't show new pr button when page is not compare pull (#26431)
Before:
After:
(TestOrg:test is a tag not branch)
Problem:
In the template, we will not add `compare pull` class when
`PageIsComparePull` is false.
https://github.com/go-gitea/gitea/blob/a370efc13f0e1ea309e324639832832bc14cb6dc/templates/repo/diff/compare.tmpl#L2
But in the js, we are using `.repository.compare.pull` to find the
button:
https://github.com/go-gitea/gitea/blob/a370efc13f0e1ea309e324639832832bc14cb6dc/web_src/js/features/repo-legacy.js#L552-L563
So, if `PageIsComparePull` is false, the `New Pull Request` button will
be there, but has no response when we click it.
Diffstat (limited to 'templates/repo')
| -rw-r--r-- | templates/repo/diff/compare.tmpl | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/templates/repo/diff/compare.tmpl b/templates/repo/diff/compare.tmpl index b4d96c3168..15574ad988 100644 --- a/templates/repo/diff/compare.tmpl +++ b/templates/repo/diff/compare.tmpl @@ -177,7 +177,7 @@ </div> {{if .IsNothingToCompare}} - {{if and $.IsSigned $.AllowEmptyPr (not .Repository.IsArchived)}} + {{if and $.IsSigned $.AllowEmptyPr (not .Repository.IsArchived) .PageIsComparePull}} <div class="ui segment">{{ctx.Locale.Tr "repo.pulls.nothing_to_compare_and_allow_empty_pr"}}</div> <div class="ui info message show-form-container {{if .Flash}}gt-hidden{{end}}"> <button class="ui button primary show-form">{{ctx.Locale.Tr "repo.pulls.new"}}</button> @@ -185,8 +185,10 @@ <div class="pullrequest-form {{if not .Flash}}gt-hidden{{end}}"> {{template "repo/issue/new_form" .}} </div> - {{else}} + {{else if and .HeadIsBranch .BaseIsBranch}} <div class="ui segment">{{ctx.Locale.Tr "repo.pulls.nothing_to_compare"}}</div> + {{else}} + <div class="ui segment">{{ctx.Locale.Tr "repo.pulls.nothing_to_compare_have_tag"}}</div> {{end}} {{else if and .PageIsComparePull (gt .CommitCount 0)}} {{if .HasPullRequest}} @@ -229,7 +231,7 @@ {{end}} {{$showDiffBox = true}} {{end}} - {{else}} + {{else if not .IsNothingToCompare}} {{$showDiffBox = true}} {{end}} </div> |