Remove the custom layout css from the terms page

This uses bootstrap utilities where necessary. It also removes the
terms page from a padding override, which was then in turn overridden
by a padding class applied to the form!

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index 018f36217f130342b458cb91563033acf28b5e98..607ea4bf6e47ea5ed2f7856c160a3d2e17852b38 100644 (file)
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -1405,8 +1405,7 @@ tr.turn:hover {
 /* Overrides for pages that use new layout conventions */
 
 .users-new,
 /* Overrides for pages that use new layout conventions */
 
 .users-new,

-.users-create,
-.users-terms {
+.users-create {

   .content-body .content-inner {
     padding: 0;
 
   .content-body .content-inner {
     padding: 0;
 

diff --git a/app/views/users/terms.html.erb b/app/views/users/terms.html.erb
index 97c1c7bf78c9ac8ec5d17129f31403cb9fb71d7b..243ce2863042040d306de0ca8867fcda1deecd58 100644 (file)
--- a/app/views/users/terms.html.erb
+++ b/app/views/users/terms.html.erb
@@ -7,13 +7,13 @@
   <div class='header-illustration new-user-terms'></div>
 <% end %>
 
   <div class='header-illustration new-user-terms'></div>
 <% end %>
 

-<%= form_tag({ :action => "save" }, { :class => " inner22 standard-form fillL" }) do %>
+<%= form_tag({ :action => "save" }, { :class => "standard-form fillL" }) do %>

   <!-- legale is <%= @legale %> -->
   <p class="deemphasize"><%= t ".read and accept with tou" %></p>
   <!-- legale is <%= @legale %> -->
   <p class="deemphasize"><%= t ".read and accept with tou" %></p>

-  <label class="standard-label">
+  <h4>

     <%= t ".heading_ct" %>
     <%= t ".heading_ct" %>

-  </label>
-  <div class='form-row horizontal-list'>
+  </h4>
+  <div class='form-row horizontal-list clearfix'>

     <p class="deemphasize"><%= t ".contributor_terms_explain" %></p>
     <label class="standard-label">
       <%= t ".legale_select" %>
     <p class="deemphasize"><%= t ".contributor_terms_explain" %></p>
     <label class="standard-label">
       <%= t ".legale_select" %>


@@ -29,7 +29,7 @@
     <% end %>
   </div>
 
     <% end %>
   </div>
 

-  <div id="contributorTerms" class="col12 legale">
+  <div id="contributorTerms" class="legale">

     <%= render :partial => "terms" %>
   </div>
 
     <%= render :partial => "terms" %>
   </div>
 


@@ -47,9 +47,9 @@
     </label>
   </div>
 
     </label>
   </div>
 

-  <label class="standard-label">
+  <h4>

     <%= t "layouts.tou" %>
     <%= t "layouts.tou" %>

-  </label>
+  </h4>

   <p class="deemphasize"><%= t ".tou_explain_html", :tou_link => link_to(t("layouts.tou"), "https://wiki.osmfoundation.org/wiki/Terms_of_Use", :target => :new) %></p>
   <div class="form-row">
     <label for="read_tou">
   <p class="deemphasize"><%= t ".tou_explain_html", :tou_link => link_to(t("layouts.tou"), "https://wiki.osmfoundation.org/wiki/Terms_of_Use", :target => :new) %></p>
   <div class="form-row">
     <label for="read_tou">


@@ -59,7 +59,7 @@
 
     <%= hidden_field_tag("referer", h(params[:referer])) unless params[:referer].nil? %>
 
 
     <%= hidden_field_tag("referer", h(params[:referer])) unless params[:referer].nil? %>
 

-    <div class="buttons form-row inner20 clearfix">
+    <div class="buttons form-row py-3 clearfix">

       <%= submit_tag("Continue", :name => "continue", :id => "continue", :disabled => true) %>
       <%= submit_tag("Cancel", :name => "decline", :id => "decline") %>
     </div>
       <%= submit_tag("Continue", :name => "continue", :id => "continue", :disabled => true) %>
       <%= submit_tag("Cancel", :name => "decline", :id => "decline") %>
     </div>