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

diff --combined app/helpers/user_helper.rb
index 4f5d7e0c2d4d52b10a6f5aa227d8c9c5c4f668f6,f48de175408b3ec475f4704c2a4f11cf8f177194..d767cb887c4730612c991a68f2a86f17d3796150
--- 1/app/helpers/user_helper.rb
--- 2/app/helpers/user_helper.rb
+++ b/app/helpers/user_helper.rb
@@@ -53,7 -53,7 +53,7 @@@ module UserHelpe
    # 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,10 -61,10 +61,10 @@@
        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 p-2 d-block",
+       :class => "auth_button btn btn-light mx-1 p-2 d-block",
        :title => t("application.auth_providers.#{name}.title")
      )
    end
@@@ -74,10 -74,10 +74,10 @@@
        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 fs-6 border rounded text-body-secondary text-decoration-none py-2 px-4 d-flex justify-content-center align-items-center",
+       :class => "auth_button btn btn-outline-secondary fs-6 border rounded py-2 px-4 d-flex justify-content-center align-items-center",
        :title => t("application.auth_providers.#{name}.title")
      )
    end

diff --combined app/views/application/_auth_providers.html.erb
index e92c9caac75d6ef65dbb8321b181ae3eee97be3f,c3fef47d9608b433dabfce8ac991a2ac240660ac..be921ee9cfdf84bf837de36dae9a0d0614b5d973
--- 1/app/views/application/_auth_providers.html.erb
--- 2/app/views/application/_auth_providers.html.erb
+++ b/app/views/application/_auth_providers.html.erb
@@@ -25,13 -25,13 +25,13 @@@
        <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"),
-                   :class => "p-2 d-block" %>
+                   :class => "btn btn-light mx-1 p-2 d-block" %>
  
        <% %w[google facebook microsoft github wikipedia].each do |provider| %>
          <% unless @preferred_auth_provider == provider %>

diff --combined test/helpers/user_helper_test.rb
index ed05aca36dffc002447c8938837e6ad7fed826ee,7d9d558f2e806d4123f3823952b42054594355b8..c7c63d62abebbf9ff8f83a521241f765c431ea52
--- 1/test/helpers/user_helper_test.rb
--- 2/test/helpers/user_helper_test.rb
+++ b/test/helpers/user_helper_test.rb
@@@ -116,8 -116,8 +116,8 @@@ class UserHelperTest < ActionView::Test
  
    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 p-2 d-block\" title=\"Log in with Google\" rel=\"nofollow\" data-method=\"post\" href=\"/auth/google\">#{img_tag}</a>", button)
+     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
  
    private