Skip to content

Commit search appearence fixes #1254

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 6, 2017
Merged

Commit search appearence fixes #1254

merged 3 commits into from
Apr 6, 2017

Conversation

pgaskin
Copy link
Contributor

@pgaskin pgaskin commented Mar 14, 2017

Fix #1238

@pgaskin pgaskin mentioned this pull request Mar 14, 2017
Closed
7 tasks
@Fastidious
Copy link

I would really love this to be fixed, but is adding   the right solution? Can't it be done with CSS, perhaps adding padding, or something of the like?

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 14, 2017
@lunny lunny added this to the 1.2.0 milestone Mar 15, 2017
@lunny lunny added type/bug topic/ui Change the appearance of the Gitea UI labels Mar 15, 2017
@strk
Copy link
Member

strk commented Mar 19, 2017

LGTM anyway, &nbsp can be fixed later :)

@tboerger tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 19, 2017
@lunny
Copy link
Member

lunny commented Mar 23, 2017

conflicted, otherwise LGTM

@tboerger tboerger 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 Mar 23, 2017
@andreynering
Copy link
Contributor

LGTM

@pgaskin
Copy link
Contributor Author

pgaskin commented Mar 29, 2017

Thanks @andreynering , I have been busy recently.

@lunny
Copy link
Member

lunny commented Mar 31, 2017

@geek1011 could make an empty force push to this PR to let drone work?

@lunny lunny merged commit d800305 into go-gitea:master Apr 6, 2017
@pgaskin pgaskin deleted the patch-6 branch May 24, 2017 01:11
@snyk-bot snyk-bot mentioned this pull request Nov 18, 2020
Open
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@appleboy appleboy appleboy approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
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 type/bug
Projects
None yet
Milestone
1.2.0
Development

Successfully merging this pull request may close these issues.

Search commits cosmetic
8 participants
@pgaskin @Fastidious @strk @lunny @andreynering @appleboy @tboerger @geek1011