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

Improve project/milestone list #24655

Closed
wants to merge 3 commits into from

Conversation

yp05327
Copy link
Contributor

@yp05327 yp05327 commented May 11, 2023

Project list
Before:
image
After:
image

Milestone List
Before:
image
After:
image

ps: It seems that project list is a copy of milestone list, and we have made some changes to milestone list.
Then we need to sync the changes to project list.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 11, 2023
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 11, 2023
@yp05327 yp05327 added the topic/ui Change the appearance of the Gitea UI label May 11, 2023
@yp05327
Copy link
Contributor Author

yp05327 commented May 11, 2023

image
The icons here seems strange 😕
Maybe we need also fix it in milestone list.

@@ -38,7 +38,11 @@
<div class="milestone list">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's a blatant copy as seen here. Possible to fix these to projects? CSS may need adjustment.

@silverwind
Copy link
Member

image The icons here seems strange 😕 Maybe we need also fix it in milestone list.

Yes, vertical center via a wrapper element possibly and gt-df gt-ac

@wxiaoguang
Copy link
Contributor

image The icons here seems strange 😕 Maybe we need also fix it in milestone list.

Yes, vertical center via a wrapper element possibly and gt-df gt-ac

After #24476 , the svg icons should be display:inline-block + vertical-align:middle , by default.

@yp05327 yp05327 changed the title Improve project list ui Improve project/milestone list May 11, 2023
@pull-request-size pull-request-size bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 11, 2023
@silverwind
Copy link
Member

silverwind commented May 11, 2023

display:inline-block + vertical-align:middle

I recall this is not a general solution and may still cause misalignment depending on icon size, text size and line height. There simply seems to be no universal way to vertically center a SVG with text in 100% of cases.

@wxiaoguang
Copy link
Contributor

may still cause misalignment depending on icon size

Do the best to use the same icon side as the font size everywhere. That's why font-awesome could win in such case.

@yp05327 yp05327 force-pushed the improve-project-name-size branch from 29bb12f to 6192b9f Compare May 11, 2023 06:35
@yp05327 yp05327 force-pushed the improve-project-name-size branch from 6192b9f to af99b5f Compare May 11, 2023 06:38
@yp05327
Copy link
Contributor Author

yp05327 commented May 11, 2023

image The icons here seems strange 😕 Maybe we need also fix it in milestone list.

Yes, vertical center via a wrapper element possibly and gt-df gt-ac

I fixed this by use gt-ml-3. Maybe it is not the best/correct way.
@silverwind @wxiaoguang
You are welcome to push commits/PR to fix it in the best/correct way. 😄

ps: the word size is 19, so I also adjusted the svg size.
image

@silverwind
Copy link
Member

silverwind commented May 11, 2023

ps: the word size is 19, so I also adjusted the svg size.

It looks still somewhat broken, the issue icon is noticeable smaller than the calendar icon. Should check how issue list renders its icons, we don't have such issues there I suppose.

I will check it later and likely push some fixes.

@yp05327
Copy link
Contributor Author

yp05327 commented Jul 14, 2023

It seems that this is fixed in #25315

@yp05327 yp05327 closed this Jul 14, 2023
@yp05327 yp05327 deleted the improve-project-name-size branch July 20, 2023 01:48
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Oct 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants