]> git.openstreetmap.org Git - rails.git/commitdiff
Arrange button sizes to default size of 36px
authorEmin Kocan <kocanmn.dev@gmail.com>
Thu, 20 Jun 2024 19:18:32 +0000 (21:18 +0200)
committerEmin Kocan <kocanmn.dev@gmail.com>
Fri, 21 Jun 2024 10:08:58 +0000 (12:08 +0200)
As discussed in #4773 there was not any clarification as why auth button sizes were reduced so 36px is now accepted as default. This commit addresses that comment/issue.

app/helpers/user_helper.rb
app/views/application/_auth_providers.html.erb
test/helpers/user_helper_test.rb

index e2de9934f6a5131150debf676c89265902b13676..4f5d7e0c2d4d52b10a6f5aa227d8c9c5c4f668f6 100644 (file)
@@ -53,7 +53,7 @@ module UserHelper
   # External authentication support
 
   def openid_logo
   # 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 = {})
   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",
       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",
       auth_path(options.merge(:provider => provider)),
       :method => :post,
       :class => "auth_button 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",
       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",
       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",
index 0cbb529a500c70a34be6dc605dcfc3d469328fdc..12f8c4d7dbddb6903584904368426f2375b870ea 100644 (file)
@@ -27,7 +27,7 @@
 
       <%= link_to image_tag("openid.png",
                             :alt => t("application.auth_providers.openid.title"),
 
       <%= link_to image_tag("openid.png",
                             :alt => t("application.auth_providers.openid.title"),
-                            :size => "24"),
+                            :size => "36"),
                   "#",
                   :id => "openid_open_url",
                   :title => t("application.auth_providers.openid.title"),
                   "#",
                   :id => "openid_open_url",
                   :title => t("application.auth_providers.openid.title"),
index 3cd53e16996a87bbe17d070d8de7c7908f2ddf22..ed05aca36dffc002447c8938837e6ad7fed826ee 100644 (file)
@@ -116,7 +116,7 @@ class UserHelperTest < ActionView::TestCase
 
   def test_auth_button
     button = auth_button("google", "google")
 
   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)
   end
 
     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)
   end