Skip to content

Conversation

@silverwind
Copy link
Member

@silverwind silverwind commented Jan 29, 2020

Fixed semi-transparent pixels of apple-touch-icon.png.

I had to manually exclude public/img/loading.png from the commit because it's an APNG and one of the tools destroys the animation.

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 29, 2020
@zeripath zeripath added the topic/ui Change the appearance of the Gitea UI label Jan 29, 2020
@zeripath zeripath added this to the 1.12.0 milestone Jan 29, 2020
Fixed semi-transparent pixels of apple-touch-icon.png.

I had to manually exclude public/img/loading.png from the commit because
it's an APNG and one of the tools destroys the animation.
@6543
Copy link
Member

6543 commented Jan 29, 2020

Why delete public/img/telegram.png ...?

@jolheiser
Copy link
Member

Why delete public/img/telegram.png ...?

It wasn't deleted, just got smaller.

@silverwind
Copy link
Member Author

Nothing is deleted here, just size-optimized.

APNG issue is google/zopfli#118, by the way.

@6543
Copy link
Member

6543 commented Jan 29, 2020

Ok, github bin diff view is missleading

@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 Jan 29, 2020
@codecov-io
Copy link

codecov-io commented Jan 29, 2020

Codecov Report

Merging #10065 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #10065      +/-   ##
==========================================
- Coverage   42.35%   42.31%   -0.05%     
==========================================
  Files         618      618              
  Lines       80774    80850      +76     
==========================================
- Hits        34213    34208       -5     
- Misses      42352    42433      +81     
  Partials     4209     4209
Impacted Files Coverage Δ
cmd/doctor.go 0% <0%> (ø) ⬆️
modules/options/dynamic.go 68.62% <0%> (-2.81%) ⬇️
services/pull/patch.go 66.66% <0%> (-3.15%) ⬇️
models/unit.go 37.03% <0%> (-2.47%) ⬇️
modules/log/event.go 64.61% <0%> (-1.03%) ⬇️
services/pull/pull.go 34.23% <0%> (+0.3%) ⬆️
services/pull/temp_repo.go 34.18% <0%> (+2.56%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96f68ce...dc83ca8. Read the comment docs.

@lafriks lafriks merged commit d7f4f87 into go-gitea:master Jan 30, 2020
@6543
Copy link
Member

6543 commented Jan 30, 2020

@NiTRoeSE can you confirm it is working now?

@silverwind silverwind deleted the img branch January 30, 2020 07:27
@NiTRoeSE
Copy link
Contributor

@NiTRoeSE can you confirm it is working now?

Yes i can confirm, looks good now.

@silverwind @6543 @zeripath

Thanks for your work.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
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.

9 participants