Use Bootstrap inline lists for <dl>s on user pages

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index 779fbbe35d0fbe705c308258ce30b104dabe1cb4..b8db4d31b6bd1d348ebb2f5d8b4cd0dd6645be24 100644 (file)
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -1251,17 +1251,6 @@ div.secondary-actions {
   text-align: center;
 }
 
   text-align: center;
 }
 

-/* Create a single-line dl */
-
-dl.dl-inline {
-  dt, dd {
-    display: inline-block;
-  }
-  dd {
-    margin-right: 1em;
-  }
-}
-

 /* Rules for OpenID logo */
 
 .openid_logo {
 /* Rules for OpenID logo */
 
 .openid_logo {

diff --git a/app/views/users/show.html.erb b/app/views/users/show.html.erb
index 4dcc9ab0546188c5a3bbed7b0128dcca7d347bf1..81e33dddb87877070c5192f3b3f4b8bb97ca1ca0 100644 (file)
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@ -118,12 +118,12 @@
 
       <div class='text-muted'>
         <small>
 
       <div class='text-muted'>
         <small>

-          <dl class="dl-inline">
-            <dt><%= t ".mapper since" %></dt>
-            <dd><%= l @user.created_at.to_date, :format => :long %></dd>
+          <dl class="list-inline">
+            <dt class="list-inline-item m-0"><%= t ".mapper since" %></dt>
+            <dd class="list-inline-item"><%= l @user.created_at.to_date, :format => :long %></dd>

             <% unless @user.terms_agreed %>
             <% unless @user.terms_agreed %>

-              <dt><%= t ".ct status" %></dt>
-              <dd>
+              <dt class="list-inline-item m-0"><%= t ".ct status" %></dt>
+              <dd class="list-inline-item">

                 <% if @user.terms_seen? -%>
                   <%= t ".ct declined" %>
                 <% else -%>
                 <% if @user.terms_seen? -%>
                   <%= t ".ct declined" %>
                 <% else -%>


@@ -188,17 +188,17 @@
       <% if current_user and current_user.administrator? -%>
         <div class='text-muted'>
           <small>
       <% if current_user and current_user.administrator? -%>
         <div class='text-muted'>
           <small>

-            <dl class='dl-inline'>
-              <dt><%= t ".email address" %></dt>
-              <dd><%= @user.email %></dd>
+            <dl class="list-inline">
+              <dt class="list-inline-item m-0"><%= t ".email address" %></dt>
+              <dd class="list-inline-item"><%= @user.email %></dd>

               <% unless @user.creation_ip.nil? -%>
               <% unless @user.creation_ip.nil? -%>

-                <dt><%= t ".created from" %></dt>
-                <dd><%= @user.creation_ip %></dd>
+                <dt class="list-inline-item m-0"><%= t ".created from" %></dt>
+                <dd class="list-inline-item"><%= @user.creation_ip %></dd>

               <% end -%>
               <% end -%>

-              <dt><%= t ".status" %></dt>
-              <dd><%= @user.status.capitalize %></dd>
-              <dt><%= t ".spam score" %></dt>
-              <dd><%= @user.spam_score %></dd>
+              <dt class="list-inline-item m-0"><%= t ".status" %></dt>
+              <dd class="list-inline-item"><%= @user.status.capitalize %></dd>
+              <dt class="list-inline-item m-0"><%= t ".spam score" %></dt>
+              <dd class="list-inline-item"><%= @user.spam_score %></dd>

             </dl>
           </small>
         </div>
             </dl>
           </small>
         </div>