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

CSS: Stronger colors for diffs #97

Merged
merged 1 commit into from
Nov 7, 2016
Merged

CSS: Stronger colors for diffs #97

merged 1 commit into from
Nov 7, 2016

Conversation

andreynering
Copy link
Contributor

It looked a too lightier for me

Before

After

@andreynering andreynering added the topic/ui Change the appearance of the Gitea UI label Nov 6, 2016
@codecov-io
Copy link

codecov-io commented Nov 6, 2016

Current coverage is 2.24% (diff: 100%)

Merging #97 into master will not change coverage

@@            master       #97   diff @@
========================================
  Files           32        32          
  Lines         7521      7521          
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           169       169          
  Misses        7335      7335          
  Partials        17        17          

Powered by Codecov. Last update 92c48da...ef5fc3c

@bkcsoft
Copy link
Member

bkcsoft commented Nov 7, 2016

did you verify that the duplication isn't needed anymore?

@tboerger tboerger added this to the 1.0.0 milestone Nov 7, 2016
@tboerger
Copy link
Member

tboerger commented Nov 7, 2016

From the screenshots it LGTM but I have not tested it.

@andreynering
Copy link
Contributor Author

andreynering commented Nov 7, 2016

@bkcsoft Yes. Actually, there's more cleanup that can be done (I didn't do because it's not related to this change).

@DblK
Copy link
Member

DblK commented Nov 7, 2016

LGTM

@DblK DblK added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 7, 2016
@tboerger tboerger removed the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 7, 2016
@tboerger
Copy link
Member

tboerger commented Nov 7, 2016

Don't add that label manually, that should be done by LGTM later on :)
@bkcsoft if you are fine with it merge it.

@lunny lunny merged commit 1b962ba into go-gitea:master Nov 7, 2016
@andreynering andreynering deleted the gitea/diff-colors branch November 7, 2016 12:39
@tboerger tboerger added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 29, 2016
ethantkoenig pushed a commit to ethantkoenig/gitea that referenced this pull request Jan 5, 2017
…gle-analytics-to-templates

Fixes go-gitea#80: add Google Analytics to templates
lunny pushed a commit to lunny/gitea that referenced this pull request Feb 7, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
tyroneyeh added a commit to tyroneyeh/gitea that referenced this pull request Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants