From 73fb245fdaf461c084713fba600b982bcb529238 Mon Sep 17 00:00:00 2001
From: Stas Vilchik <vilchiks@gmail.com>
Date: Tue, 2 Feb 2016 16:52:25 +0100
Subject: [PATCH] improve rendering of login form with many oauth providers

---
 .../sonar-web/src/main/less/pages/login.less  |  3 +-
 .../WEB-INF/app/views/sessions/_form.html.erb | 32 ++++++++++---------
 2 files changed, 19 insertions(+), 16 deletions(-)

diff --git a/server/sonar-web/src/main/less/pages/login.less b/server/sonar-web/src/main/less/pages/login.less
index 3af71a8b5ef..15f37875df4 100644
--- a/server/sonar-web/src/main/less/pages/login.less
+++ b/server/sonar-web/src/main/less/pages/login.less
@@ -70,7 +70,6 @@
 }
 
 .oauth-providers {
-  margin-bottom: 30px;
   padding-bottom: 30px;
   border-bottom: 1px solid @barBorderColor;
 
@@ -80,6 +79,8 @@
     flex-wrap: wrap;
 
     & > li {
+      margin-bottom: 30px;
+
       & > a {
         display: block;
         width: 180px;
diff --git a/server/sonar-web/src/main/webapp/WEB-INF/app/views/sessions/_form.html.erb b/server/sonar-web/src/main/webapp/WEB-INF/app/views/sessions/_form.html.erb
index 556822c9e87..d0b72146d04 100644
--- a/server/sonar-web/src/main/webapp/WEB-INF/app/views/sessions/_form.html.erb
+++ b/server/sonar-web/src/main/webapp/WEB-INF/app/views/sessions/_form.html.erb
@@ -1,20 +1,22 @@
 <h1 class="maintenance-title text-center">Log In to SonarQube</h1>
 
-<section class="oauth-providers">
-  <% auth_providers = Api::Utils.java_facade.getIdentityProviders().to_a %>
-  <ul>
-    <% auth_providers.each do |provider| %>
-      <li>
-        <a href="<%= ApplicationController.root_context -%>/sessions/init/<%= provider.getKey().to_s %>"
-           title="Log in with <%= provider.getName().to_s -%>">
-          <img alt="<%= provider.getName().to_s -%>" width="20" height="20"
-               src="<%= ApplicationController.root_context + provider.getIconPath().to_s -%>">
-          <span>Log in with <%= provider.getName().to_s -%></span>
-        </a>
-      </li>
-    <% end %>
-  </ul>
-</section>
+<% auth_providers = Api::Utils.java_facade.getIdentityProviders().to_a %>
+<% if auth_providers.size > 0 %>
+  <section class="oauth-providers">
+    <ul>
+      <% auth_providers.each do |provider| %>
+        <li>
+          <a href="<%= ApplicationController.root_context -%>/sessions/init/<%= provider.getKey().to_s %>"
+             title="Log in with <%= provider.getName().to_s -%>">
+            <img alt="<%= provider.getName().to_s -%>" width="20" height="20"
+                 src="<%= ApplicationController.root_context + provider.getIconPath().to_s -%>">
+            <span>Log in with <%= provider.getName().to_s -%></span>
+          </a>
+        </li>
+      <% end %>
+    </ul>
+  </section>
+<% end %>
 
 <form id="login_form" action="<%= url_for :controller => 'sessions', :action => 'login' -%>" method="post">
   <input type="hidden" name="return_to_anchor" value="<%= h @return_to_anchor %>">
-- 
2.39.5