-
-
Notifications
You must be signed in to change notification settings - Fork 6k
Closed
Closed
Copy link
Labels
issue/confirmedIssue has been reviewed and confirmed to be present or accepted to be implementedIssue has been reviewed and confirmed to be present or accepted to be implementedtopic/uiChange the appearance of the Gitea UIChange the appearance of the Gitea UI
Description
I think the current implementation takes away too much space from the important left-aligned column because having the buttons inside individual fixed columns introduces unnecessary spacing. I'd put all three buttons in the same td
and make them look like this (remove compact
class on the pull button):
Then right-align the diffstat and let the leftmost column take the remaining space.
Originally posted by @silverwind in #7520 (comment)
Metadata
Metadata
Assignees
Labels
issue/confirmedIssue has been reviewed and confirmed to be present or accepted to be implementedIssue has been reviewed and confirmed to be present or accepted to be implementedtopic/uiChange the appearance of the Gitea UIChange the appearance of the Gitea UI
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
6543 commentedon Jul 24, 2019
blocked by #7602 #7604
silverwind commentedon Jul 25, 2019
Not sure if that split-up was really necessary. Those two PRs conflict each other unfortunately. Maybe wait till #7602 is merged and then do the remaining changes in #7604 to achieve the equivalent of that screenshot?
6543 commentedon Jul 25, 2019
i thought the contributor guidline tells me that if i like to get changes merged quik i should make smal PRs wich a single wish/topic
6543 commentedon Jul 25, 2019
i cann still klose the smal PRs and contribute to the big one?
silverwind commentedon Jul 25, 2019
Yes, small changes get merged faster but your old one wasn't really that big. Anyways, I'd suggest sticking to the plan above. #7602 should probably be landed by tomorrow.
6543 commentedon Jul 25, 2019
thx
stale commentedon Sep 23, 2019
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.
6543 commentedon Oct 14, 2019
@silverwind ask for reopen this it you like - i curently dont work on this ...
6543 commentedon Sep 17, 2020
close this since this is no huge deal and if somebody came with good idears ... he can make it's own issue or pull