Skip to content
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

Commit status list squished on commit view #18498

Closed
parnic-sks opened this issue Jan 31, 2022 · 6 comments · Fixed by #18538 or #18661
Closed

Commit status list squished on commit view #18498

parnic-sks opened this issue Jan 31, 2022 · 6 comments · Fixed by #18538 or #18661
Labels
issue/stale topic/ui Change the appearance of the Gitea UI type/bug
Milestone

Comments

@parnic-sks
Copy link

Gitea Version

1.16.0

Operating System

Windows 11

Browser Version

Google Chrome Version 97.0.4692.99 (Official Build) (64-bit)

Can you reproduce the bug on the Gitea demo site?

Yes

Description

Clicking on the status icon after the commit summary on a commit view page shows a squished box that is very difficult to use. This worked fine in 1.15.x and broke in 1.16.0

Broken: https://try.gitea.io/parnic-sks/commit-status-test/commit/5333d3d91b0fabe51286fa8bbe2d996901ca60e6

It works fine in other locations, however:

Screenshots

Broken:
image

Correct:
image

@lunny lunny added type/bug topic/ui Change the appearance of the Gitea UI labels Feb 1, 2022
@lunny lunny added this to the 1.16.1 milestone Feb 1, 2022
@zeripath
Copy link
Contributor

zeripath commented Feb 1, 2022

From my testing it appears to be related to .ui.attached.header .right(not:dropdown) css selector setting the height to 30px;

If we add a height: auto then this problem goes away.

zeripath added a commit to zeripath/gitea that referenced this issue Feb 1, 2022
When viewing commits as commits the commit-status box will be fixed at 30px in height
due to being forced to be this size by a fomantic selector. This PR simply adds a
few more selectors to force this to have height auto.

Fix go-gitea#18498

Signed-off-by: Andrew Thornton <art27@cantab.net>
lunny pushed a commit that referenced this issue Feb 4, 2022
* Ensure commit-statuses box is sized correctly in headers

When viewing commits as commits the commit-status box will be fixed at 30px in height
due to being forced to be this size by a fomantic selector. This PR simply adds a
few more selectors to force this to have height auto.

Fix #18498

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Update web_src/less/_repository.less

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
zeripath added a commit to zeripath/gitea that referenced this issue Feb 4, 2022
)

Backport go-gitea#18538
Backport go-gitea#18605

* Ensure commit-statuses box is sized correctly in headers

When viewing commits as commits the commit-status box will be fixed at 30px in height
due to being forced to be this size by a fomantic selector. This PR simply adds a
few more selectors to force this to have height auto.

Fix go-gitea#18498

Signed-off-by: Andrew Thornton <art27@cantab.net>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
6543 pushed a commit that referenced this issue Feb 4, 2022
…8606)

* Ensure commit-statuses box is sized correctly in headers (#18538)

Backport #18538
Backport #18605

* Ensure commit-statuses box is sized correctly in headers

When viewing commits as commits the commit-status box will be fixed at 30px in height
due to being forced to be this size by a fomantic selector. This PR simply adds a
few more selectors to force this to have height auto.

Fix #18498

Signed-off-by: Andrew Thornton <art27@cantab.net>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>

* Remove the spurious space in the .ui.right additional selector

Somehow a spurious space sneaked in to #18538
this PR simply removes it.

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@parnic-sks
Copy link
Author

@zeripath On 1.16.1, the box is the right size now, but the "Details" links on the right side are all stacked on top of one another:

image

This is visible in the try.gitea.io link in the description of this issue: https://try.gitea.io/parnic-sks/commit-status-test/commit/5333d3d91b0fabe51286fa8bbe2d996901ca60e6

@wxiaoguang
Copy link
Contributor

It's caused by the postion: absolute:

image

@wxiaoguang wxiaoguang reopened this Feb 7, 2022
@zeripath
Copy link
Contributor

zeripath commented Feb 7, 2022

OK

diff --git a/templates/repo/commit_statuses.tmpl b/templates/repo/commit_statuses.tmpl
index d2e9f0bd1..f33635abf 100644
--- a/templates/repo/commit_statuses.tmpl
+++ b/templates/repo/commit_statuses.tmpl
@@ -2,11 +2,11 @@
 <div class="ui popup very wide fixed basic commit-statuses">
 	<div class="ui relaxed list divided">
 		{{range .Statuses}}
-			<div class="ui item singular-status">
+			<div class="ui item singular-status df">
 				<span>{{template "repo/commit_status" .}}</span>
-				<span class="ui">{{.Context}} <span class="text grey">{{.Description}}</span></span>
+				<span class="ui f1">{{.Context}} <span class="text grey">{{.Description}}</span></span>
 				{{if .TargetURL}}
-					<div class="ui right"><a href="{{.TargetURL}}" target="_blank" rel="noopener noreferrer">{{$.root.i18n.Tr "repo.pulls.status_checks_details"}}</a></div>
+					<div class="ui"><a href="{{.TargetURL}}" target="_blank" rel="noopener noreferrer">{{$.root.i18n.Tr "repo.pulls.status_checks_details"}}</a></div>
 				{{end}}
 			</div>
 		{{end}}

@wxiaoguang
Copy link
Contributor

Wait for backport

@wxiaoguang wxiaoguang reopened this Feb 11, 2022
Chianina pushed a commit to Chianina/gitea that referenced this issue Mar 28, 2022
)

* Ensure commit-statuses box is sized correctly in headers

When viewing commits as commits the commit-status box will be fixed at 30px in height
due to being forced to be this size by a fomantic selector. This PR simply adds a
few more selectors to force this to have height auto.

Fix go-gitea#18498

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Update web_src/less/_repository.less

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@stale
Copy link

stale bot commented Apr 17, 2022

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

@stale stale bot added the issue/stale label Apr 17, 2022
@wxiaoguang wxiaoguang modified the milestones: 1.16.1, 1.16.2 Apr 17, 2022
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/stale topic/ui Change the appearance of the Gitea UI type/bug
Projects
None yet
4 participants