From be745be0a4f3feab01f896f1c0f17164bb30234b Mon Sep 17 00:00:00 2001 From: silverwind Date: Sat, 22 May 2021 03:18:43 +0200 Subject: [PATCH] Double the avatar size factor (#15941) * Double the avatar size factor This results on finer Avatar rendering on Hi-DPI display. * fix test Co-authored-by: zeripath Co-authored-by: techknowlogick --- models/avatar.go | 2 +- modules/repository/commits_test.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/models/avatar.go b/models/avatar.go index 4f02fe508981..b4c078f8cf9f 100644 --- a/models/avatar.go +++ b/models/avatar.go @@ -44,7 +44,7 @@ const DefaultAvatarSize = -1 const DefaultAvatarPixelSize = 28 // AvatarRenderedSizeFactor is the factor by which the default size is increased for finer rendering -const AvatarRenderedSizeFactor = 2 +const AvatarRenderedSizeFactor = 4 // HashEmail hashes email address to MD5 string. // https://en.gravatar.com/site/implement/hash/ diff --git a/modules/repository/commits_test.go b/modules/repository/commits_test.go index 16677fe8a684..ba69a7f97ca7 100644 --- a/modules/repository/commits_test.go +++ b/modules/repository/commits_test.go @@ -112,13 +112,13 @@ func TestPushCommits_AvatarLink(t *testing.T) { pushCommits.Len = len(pushCommits.Commits) assert.Equal(t, - "https://secure.gravatar.com/avatar/ab53a2911ddf9b4817ac01ddcd3d975f?d=identicon&s=56", + "https://secure.gravatar.com/avatar/ab53a2911ddf9b4817ac01ddcd3d975f?d=identicon&s=112", pushCommits.AvatarLink("user2@example.com")) assert.Equal(t, "https://secure.gravatar.com/avatar/"+ fmt.Sprintf("%x", md5.Sum([]byte("nonexistent@example.com")))+ - "?d=identicon&s=56", + "?d=identicon&s=112", pushCommits.AvatarLink("nonexistent@example.com")) }