Skip to content

Remove padding/border-radius on image diffs#12346

Merged
zeripath merged 2 commits into
go-gitea:masterfrom
silverwind:imgdiff
Jul 28, 2020
Merged

Remove padding/border-radius on image diffs#12346
zeripath merged 2 commits into
go-gitea:masterfrom
silverwind:imgdiff

Conversation

@silverwind
Copy link
Copy Markdown
Member

Before:

image

After:

image

@zeripath zeripath added the topic/ui Change the appearance of the Gitea UI label Jul 28, 2020
@zeripath zeripath added this to the 1.13.0 milestone Jul 28, 2020
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jul 28, 2020
@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 Jul 28, 2020
@zeripath
Copy link
Copy Markdown
Contributor

make lg-tm work

@zeripath zeripath merged commit cd7f1fd into go-gitea:master Jul 28, 2020
@silverwind silverwind deleted the imgdiff branch July 28, 2020 16:36
silverwind added a commit to silverwind/gitea that referenced this pull request Jul 30, 2020
go-gitea#12346 was using the wrong
selector which affected only the single image view. Use the correct
selector for image diffs. Removal of border-radius is still worth to
keep to override the global 3px on <img>.
techknowlogick pushed a commit that referenced this pull request Aug 6, 2020
#12346 was using the wrong
selector which affected only the single image view. Use the correct
selector for image diffs. Removal of border-radius is still worth to
keep to override the global 3px on <img>.

Co-authored-by: Lauris BH <lauris@nix.lv>
@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.

4 participants