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

Comment Header fixes #13356

Merged
merged 2 commits into from Oct 31, 2020
Merged

Comment Header fixes #13356

merged 2 commits into from Oct 31, 2020

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Oct 29, 2020

Apply more flexboxes on comment header and remove float hacks. Needs 1.13 backport.

Fixes: #13316

Also, action icons are better centered. Before:
image

After:
image

@mrsdizzie please confirm, I could not test your case with more than two header children.

Apply more flexboxes on comment header and remove float hacks. Needs
1.13 backport.

Fixes: go-gitea#13316
@zeripath zeripath added the topic/ui Change the appearance of the Gitea UI label Oct 30, 2020
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Oct 30, 2020
@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 Oct 31, 2020
@lafriks lafriks merged commit 0e6bf7e into go-gitea:master Oct 31, 2020
@lafriks
Copy link
Member

lafriks commented Oct 31, 2020

Please send backport

silverwind added a commit to silverwind/gitea that referenced this pull request Oct 31, 2020
Apply more flexboxes on comment header and remove float hacks. Needs
1.13 backport.

Fixes: go-gitea#13316

Co-authored-by: Lauris BH <lauris@nix.lv>
techknowlogick pushed a commit that referenced this pull request Oct 31, 2020
Apply more flexboxes on comment header and remove float hacks. Needs
1.13 backport.

Fixes: #13316

Co-authored-by: Lauris BH <lauris@nix.lv>

Co-authored-by: Lauris BH <lauris@nix.lv>
@techknowlogick techknowlogick added the backport/done All backports for this PR have been created label Nov 5, 2020
@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
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with comment header display after change to flex
5 participants