Merge remote-tracking branch 'upstream/pull/4916'

diff --git a/app/assets/images/openid.png b/app/assets/images/openid.png
deleted file mode 100644 (file)
index dc7e01f..0000000
Binary files a/app/assets/images/openid.png and /dev/null differ

diff --git a/app/helpers/user_helper.rb b/app/helpers/user_helper.rb
index f48de175408b3ec475f4704c2a4f11cf8f177194..d767cb887c4730612c991a68f2a86f17d3796150 100644 (file)
--- a/app/helpers/user_helper.rb
+++ b/app/helpers/user_helper.rb
@@ -53,7 +53,7 @@ module UserHelper
   # External authentication support
 
   def openid_logo
-    image_tag "openid.svg", :size => "24", :alt => t("application.auth_providers.openid_logo_alt"), :class => "align-text-bottom"
+    image_tag "openid.svg", :size => "36", :alt => t("application.auth_providers.openid_logo_alt"), :class => "align-text-bottom"
   end
 
   def auth_button(name, provider, options = {})
@@ -61,7 +61,7 @@ module UserHelper
       image_tag("#{name}.svg",
                 :alt => t("application.auth_providers.#{name}.alt"),
                 :class => "rounded-1",
-                :size => "24"),
+                :size => "36"),
       auth_path(options.merge(:provider => provider)),
       :method => :post,
       :class => "auth_button btn btn-light mx-1 p-2 d-block",
@@ -74,7 +74,7 @@ module UserHelper
       image_tag("#{name}.svg",
                 :alt => t("application.auth_providers.#{name}.alt"),
                 :class => "rounded-1 me-3",
-                :size => "24") + t("application.auth_providers.#{name}.title"),
+                :size => "36") + t("application.auth_providers.#{name}.title"),
       auth_path(options.merge(:provider => provider)),
       :method => :post,
       :class => "auth_button btn btn-outline-secondary fs-6 border rounded py-2 px-4 d-flex justify-content-center align-items-center",

diff --git a/app/views/application/_auth_providers.html.erb b/app/views/application/_auth_providers.html.erb
index c3fef47d9608b433dabfce8ac991a2ac240660ac..be921ee9cfdf84bf837de36dae9a0d0614b5d973 100644 (file)
--- a/app/views/application/_auth_providers.html.erb
+++ b/app/views/application/_auth_providers.html.erb
@@ -25,9 +25,9 @@
       <div class="list-inline justify-content-center d-flex align-items-center flex-wrap w-100">
     <% end %>
 
-      <%= link_to image_tag("openid.png",
+      <%= link_to image_tag("openid.svg",
                             :alt => t("application.auth_providers.openid.title"),
-                            :size => "24"),
+                            :size => "36"),
                   "#",
                   :id => "openid_open_url",
                   :title => t("application.auth_providers.openid.title"),

diff --git a/test/helpers/user_helper_test.rb b/test/helpers/user_helper_test.rb
index 7d9d558f2e806d4123f3823952b42054594355b8..c7c63d62abebbf9ff8f83a521241f765c431ea52 100644 (file)
--- a/test/helpers/user_helper_test.rb
+++ b/test/helpers/user_helper_test.rb
@@ -116,7 +116,7 @@ class UserHelperTest < ActionView::TestCase
 
   def test_auth_button
     button = auth_button("google", "google")
-    img_tag = "<img alt=\"Log in with a Google OpenID\" class=\"rounded-1\" src=\"/images/google.svg\" width=\"24\" height=\"24\" />"
+    img_tag = "<img alt=\"Log in with a Google OpenID\" class=\"rounded-1\" src=\"/images/google.svg\" width=\"36\" height=\"36\" />"
     assert_equal("<a class=\"auth_button btn btn-light mx-1 p-2 d-block\" title=\"Log in with Google\" rel=\"nofollow\" data-method=\"post\" href=\"/auth/google\">#{img_tag}</a>", button)
   end