]> git.openstreetmap.org Git - rails.git/commitdiff
Bootstrap 5: Add form-label class to hand-crafted form labels
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 29 Jun 2022 16:02:43 +0000 (17:02 +0100)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 20 Jul 2022 13:41:56 +0000 (14:41 +0100)
This is done automatically for forms created by bootstrap_form

app/views/accounts/edit.html.erb
app/views/sessions/new.html.erb
app/views/user_blocks/edit.html.erb
app/views/user_blocks/new.html.erb
app/views/users/new.html.erb
app/views/users/terms.html.erb

index cd53d294d9708978028fb3a161a9b1194abb2e34..038f5c09f9810d9967fd6fd342962942bf37f4ea 100644 (file)
@@ -17,7 +17,7 @@
   <%= f.password_field :pass_crypt_confirmation, :value => "", :autocomplete => "new-password" %>
 
   <fieldset class="form-group">
   <%= f.password_field :pass_crypt_confirmation, :value => "", :autocomplete => "new-password" %>
 
   <fieldset class="form-group">
-    <label for="user_auth_provider"><%= t(".external auth") %></label>
+    <label for="user_auth_provider" class="form-label"><%= t(".external auth") %></label>
     <div class="row">
       <%= f.select(:auth_provider, Auth.providers, :hide_label => true, :wrapper => { :class => "col-auto mb-0" }) %>
       <%= f.text_field(:auth_uid, :hide_label => true, :wrapper => { :class => "col mb-0" }) %>
     <div class="row">
       <%= f.select(:auth_provider, Auth.providers, :hide_label => true, :wrapper => { :class => "col-auto mb-0" }) %>
       <%= f.text_field(:auth_uid, :hide_label => true, :wrapper => { :class => "col mb-0" }) %>
@@ -26,7 +26,7 @@
   </fieldset>
 
   <div class="form-group">
   </fieldset>
 
   <div class="form-group">
-    <label><%= t ".public editing.heading" %></label>
+    <label class="form-label"><%= t ".public editing.heading" %></label>
     <span class="form-text text-muted">
       <% if current_user.data_public? %>
         <%= t ".public editing.enabled" %>
     <span class="form-text text-muted">
       <% if current_user.data_public? %>
         <%= t ".public editing.enabled" %>
@@ -39,7 +39,7 @@
   </div>
 
   <div class="form-group">
   </div>
 
   <div class="form-group">
-    <label><%= t ".contributor terms.heading" %></label>
+    <label class="form-label"><%= t ".contributor terms.heading" %></label>
     <span class="form-text text-muted">
       <% if current_user.terms_agreed? %>
         <%= t ".contributor terms.agreed" %>
     <span class="form-text text-muted">
       <% if current_user.terms_agreed? %>
         <%= t ".contributor terms.agreed" %>
index 18f536206007ab965724be7ec2cefc7d038c7bb7..ddebc55a45448027a8e45eab3285e7044e9ef72d 100644 (file)
@@ -25,7 +25,7 @@
 
   <div id="loginForm">
     <div class="form-group">
 
   <div id="loginForm">
     <div class="form-group">
-      <label><%= t ".with external" %></label>
+      <label class="form-label"><%= t ".with external" %></label>
 
       <ul class='list-unstyled' id="login_auth_buttons">
         <li><%= link_to image_tag("openid.png", :alt => t(".auth_providers.openid.title")), "#", :id => "openid_open_url", :title => t(".auth_providers.openid.title") %></li>
 
       <ul class='list-unstyled' id="login_auth_buttons">
         <li><%= link_to image_tag("openid.png", :alt => t(".auth_providers.openid.title")), "#", :id => "openid_open_url", :title => t(".auth_providers.openid.title") %></li>
@@ -48,7 +48,7 @@
 
       <%= form_tag(auth_path(:provider => "openid"), :id => "openid_login_form") do %>
         <div id='login_openid_url' class="form-group">
 
       <%= form_tag(auth_path(:provider => "openid"), :id => "openid_login_form") do %>
         <div id='login_openid_url' class="form-group">
-          <label for='openid_url'><%= t ".openid_html", :logo => openid_logo %></label>
+          <label for='openid_url' class="form-label"><%= t ".openid_html", :logo => openid_logo %></label>
           <%= hidden_field_tag("referer", params[:referer], :autocomplete => "off") %>
           <%= text_field_tag("openid_url", "", :tabindex => 3, :autocomplete => "on", :class => "openid_url form-control") %>
           <span class="form-text text-muted">(<a href="<%= t "accounts.edit.openid.link" %>" target="_new"><%= t "accounts.edit.openid.link text" %></a>)</span>
           <%= hidden_field_tag("referer", params[:referer], :autocomplete => "off") %>
           <%= text_field_tag("openid_url", "", :tabindex => 3, :autocomplete => "on", :class => "openid_url form-control") %>
           <span class="form-text text-muted">(<a href="<%= t "accounts.edit.openid.link" %>" target="_new"><%= t "accounts.edit.openid.link text" %></a>)</span>
index 3591b9c0f97981bd647e4d45cbfdd2c7775b2f43..db7c5e057f360d5c80ceab91cdbfee6e4fe86589 100644 (file)
@@ -15,7 +15,7 @@
   <%= f.richtext_field :reason, :cols => 80, :rows => 20, :format => @user_block.reason_format %>
 
   <%= f.form_group do %>
   <%= f.richtext_field :reason, :cols => 80, :rows => 20, :format => @user_block.reason_format %>
 
   <%= f.form_group do %>
-    <%= label_tag "user_block_period", t(".period") %><br />
+    <%= label_tag "user_block_period", t(".period"), :class => "form-label" %>
     <%= select_tag("user_block_period",
                    options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period]),
                    :class => "form-select") %>
     <%= select_tag("user_block_period",
                    options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period]),
                    :class => "form-select") %>
index c1195343b8c5ae6202e9d27929009a0a032a2ef6..c390134ba3db8482127f0c0a34a085a147e39aeb 100644 (file)
@@ -9,7 +9,7 @@
   <%= f.richtext_field :reason, :cols => 80, :rows => 20 %>
 
   <%= f.form_group do %>
   <%= f.richtext_field :reason, :cols => 80, :rows => 20 %>
 
   <%= f.form_group do %>
-    <%= label_tag "user_block_period", t(".period") %><br />
+    <%= label_tag "user_block_period", t(".period"), :class => "form-label" %>
     <%= select_tag("user_block_period",
                    options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period]),
                    :class => "form-select") %>
     <%= select_tag("user_block_period",
                    options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period]),
                    :class => "form-select") %>
index 201597f4e446aa37438952388d2e1a95fabf5b6e..c53135678d052cd47d7f96e146fe34c3c90bf611 100644 (file)
@@ -26,7 +26,7 @@
       <%= f.text_field :display_name, :help => t(".display name description"), :tabindex => 3 %>
 
       <fieldset class="form-group" id="auth_field">
       <%= f.text_field :display_name, :help => t(".display name description"), :tabindex => 3 %>
 
       <fieldset class="form-group" id="auth_field">
-        <label for="user_auth_provider"><%= t(".external auth") %></label>
+        <label for="user_auth_provider" class="form-label"><%= t(".external auth") %></label>
         <div class="row">
           <%= f.select(:auth_provider, Auth.providers, :default => "", :hide_label => true, :wrapper => { :class => "col-auto mb-0" }, :tabindex => 4) %>
           <%= f.text_field(:auth_uid, :hide_label => true, :wrapper => { :class => "col mb-0" }, :tabindex => 5) %>
         <div class="row">
           <%= f.select(:auth_provider, Auth.providers, :default => "", :hide_label => true, :wrapper => { :class => "col-auto mb-0" }, :tabindex => 4) %>
           <%= f.text_field(:auth_uid, :hide_label => true, :wrapper => { :class => "col mb-0" }, :tabindex => 5) %>
index 58bddaca5fcee2ff82796486908db0bb8e14b1c7..1f42e9085c09cc90513e1b527ff723b6306b8fcb 100644 (file)
@@ -16,7 +16,7 @@
     <%= t ".heading_ct" %>
   </h4>
   <p class="text-muted"><%= t ".contributor_terms_explain" %></p>
     <%= t ".heading_ct" %>
   </h4>
   <p class="text-muted"><%= t ".contributor_terms_explain" %></p>
-  <label>
+  <label class="form-label">
     <%= t ".legale_select" %>
   </label>
   <div class="form-group">
     <%= t ".legale_select" %>
   </label>
   <div class="form-group">