Commit c7a946fae9ed5316f5ef228b60d4727e5d96af2c
Committed by
Rafael Manzo
1 parent
60db995f
Exists in
colab
and in
4 other branches
Highlighting required fields
Signed off by: Daniel Alves <danpaulalves@gmail.com>
Showing
10 changed files
with
21 additions
and
14 deletions
Show diff stats
app/assets/stylesheets/form_with_tooltip.css
app/views/compound_metric_configurations/_form.html.erb
| @@ -9,7 +9,7 @@ | @@ -9,7 +9,7 @@ | ||
| 9 | <div class="form-row"> | 9 | <div class="form-row"> |
| 10 | <div class="field-container"> | 10 | <div class="field-container"> |
| 11 | <%= f.label :code, class: 'control-label' %> | 11 | <%= f.label :code, class: 'control-label' %> |
| 12 | - <%= f.text_field :code, class: 'text-field form-control' %> | 12 | + <%= f.text_field :code, :required => true, class: 'text-field form-control' %> |
| 13 | </div> | 13 | </div> |
| 14 | <div class="help-container"> | 14 | <div class="help-container"> |
| 15 | <p> | 15 | <p> |
| @@ -25,7 +25,7 @@ | @@ -25,7 +25,7 @@ | ||
| 25 | <div class="form-row"> | 25 | <div class="form-row"> |
| 26 | <div class="field-container"> | 26 | <div class="field-container"> |
| 27 | <%= f.label :weight, class: 'control-label' %> | 27 | <%= f.label :weight, class: 'control-label' %> |
| 28 | - <%= f.text_field :weight, class: 'text-field form-control' %> | 28 | + <%= f.text_field :weight, :required => true, class: 'text-field form-control' %> |
| 29 | </div> | 29 | </div> |
| 30 | <div class="help-container"> | 30 | <div class="help-container"> |
| 31 | <p> | 31 | <p> |
app/views/compound_metric_configurations/_metric_options.html.erb
| 1 | <div class="form-row"> | 1 | <div class="form-row"> |
| 2 | <div class="field-container"> | 2 | <div class="field-container"> |
| 3 | <%= f.label :name, class: 'control-label' %> | 3 | <%= f.label :name, class: 'control-label' %> |
| 4 | + <%-#FIXME Change the following field to "required => true" after CompoundMetrics Form Bug fix -%> | ||
| 4 | <%= f.text_field :name, class: 'text-field form-control', value: (metric.name unless metric.nil?) %> | 5 | <%= f.text_field :name, class: 'text-field form-control', value: (metric.name unless metric.nil?) %> |
| 5 | </div> | 6 | </div> |
| 6 | <div class="help-container"> | 7 | <div class="help-container"> |
| @@ -25,6 +26,7 @@ | @@ -25,6 +26,7 @@ | ||
| 25 | <div class="form-row"> | 26 | <div class="form-row"> |
| 26 | <div class="field-container"> | 27 | <div class="field-container"> |
| 27 | <%= f.label :script, class: 'control-label' %> | 28 | <%= f.label :script, class: 'control-label' %> |
| 29 | + <%-#FIXME Change the following field to "required => true" after CompoundMetrics Form Bug fix -%> | ||
| 28 | <%= f.text_area :script, class: 'text-area form-control', value: (metric.script unless metric.nil?) %> | 30 | <%= f.text_area :script, class: 'text-area form-control', value: (metric.script unless metric.nil?) %> |
| 29 | </div> | 31 | </div> |
| 30 | <div class="help-container"> | 32 | <div class="help-container"> |
app/views/metric_configurations/_form.html.erb
| @@ -6,7 +6,7 @@ | @@ -6,7 +6,7 @@ | ||
| 6 | <div class="form-row"> | 6 | <div class="form-row"> |
| 7 | <div class="field-container"> | 7 | <div class="field-container"> |
| 8 | <%= f.label :weight, class: 'control-label' %> | 8 | <%= f.label :weight, class: 'control-label' %> |
| 9 | - <%= f.text_field :weight, class: 'text-field form-control' %> | 9 | + <%= f.text_field :weight, :required => true, class: 'text-field form-control' %> |
| 10 | </div> | 10 | </div> |
| 11 | <div class="help-container"> | 11 | <div class="help-container"> |
| 12 | <p> | 12 | <p> |
app/views/mezuro_configurations/_form.html.erb
| @@ -7,7 +7,7 @@ | @@ -7,7 +7,7 @@ | ||
| 7 | <div class="form-row"> | 7 | <div class="form-row"> |
| 8 | <div class="field-container"> | 8 | <div class="field-container"> |
| 9 | <%= f.label :name, class: 'control-label' %><br> | 9 | <%= f.label :name, class: 'control-label' %><br> |
| 10 | - <%= f.text_field :name, class: 'text-field form-control' %> | 10 | + <%= f.text_field :name, :required => true, class: 'text-field form-control' %> |
| 11 | </div> | 11 | </div> |
| 12 | </div> | 12 | </div> |
| 13 | 13 |
app/views/mezuro_ranges/_form.html.erb
| @@ -23,7 +23,7 @@ | @@ -23,7 +23,7 @@ | ||
| 23 | <div class="field-container" > | 23 | <div class="field-container" > |
| 24 | <%= f.label :beginning, class: 'control-label' %> | 24 | <%= f.label :beginning, class: 'control-label' %> |
| 25 | <div class="input-group"> | 25 | <div class="input-group"> |
| 26 | - <%= f.text_field :beginning, class: 'diminished-text-field form-control' %> | 26 | + <%= f.text_field :beginning, :required => true, class: 'diminished-text-field form-control' %> |
| 27 | <span class="input-group-btn"> | 27 | <span class="input-group-btn"> |
| 28 | <%= link_to '-∞'.html_safe, 'javascript:void(0)', :onClick => "jQuery( '#mezuro_range_beginning' ).val('-INF');", class: 'btn btn-default' %> | 28 | <%= link_to '-∞'.html_safe, 'javascript:void(0)', :onClick => "jQuery( '#mezuro_range_beginning' ).val('-INF');", class: 'btn btn-default' %> |
| 29 | </span> | 29 | </span> |
| @@ -41,7 +41,7 @@ | @@ -41,7 +41,7 @@ | ||
| 41 | <div class="field-container"> | 41 | <div class="field-container"> |
| 42 | <%= f.label :end, class: 'control-label' %> | 42 | <%= f.label :end, class: 'control-label' %> |
| 43 | <div class="input-group"> | 43 | <div class="input-group"> |
| 44 | - <%= f.text_field :end, class: 'diminished-text-field form-control' %> | 44 | + <%= f.text_field :end, :required => true, class: 'diminished-text-field form-control' %> |
| 45 | <span class="input-group-btn"> | 45 | <span class="input-group-btn"> |
| 46 | <%= link_to '∞'.html_safe, 'javascript:void(0)', :onClick => "jQuery( '#mezuro_range_end' ).val('INF');", class: 'btn btn-default align-button' %> | 46 | <%= link_to '∞'.html_safe, 'javascript:void(0)', :onClick => "jQuery( '#mezuro_range_end' ).val('INF');", class: 'btn btn-default align-button' %> |
| 47 | </span> | 47 | </span> |
app/views/projects/_form.html.erb
| @@ -6,7 +6,7 @@ | @@ -6,7 +6,7 @@ | ||
| 6 | <div class="form-row"> | 6 | <div class="form-row"> |
| 7 | <div class="field-container"> | 7 | <div class="field-container"> |
| 8 | <%= f.label :name, class: 'control-label' %><br> | 8 | <%= f.label :name, class: 'control-label' %><br> |
| 9 | - <%= f.text_field :name, class: 'text-field form-control' %> | 9 | + <%= f.text_field :name, :required => true, class: 'text-field form-control' %> |
| 10 | </div> | 10 | </div> |
| 11 | </div> | 11 | </div> |
| 12 | 12 |
app/views/reading_groups/_form.html.erb
| @@ -7,7 +7,7 @@ | @@ -7,7 +7,7 @@ | ||
| 7 | <div class="form-row"> | 7 | <div class="form-row"> |
| 8 | <div class="field-container"> | 8 | <div class="field-container"> |
| 9 | <%= f.label :name, class: 'control-label' %> | 9 | <%= f.label :name, class: 'control-label' %> |
| 10 | - <%= f.text_field :name, class: 'text-field form-control' %> | 10 | + <%= f.text_field :name, :required => true, class: 'text-field form-control' %> |
| 11 | </div> | 11 | </div> |
| 12 | </div> | 12 | </div> |
| 13 | 13 |
app/views/readings/_form.html.erb
| @@ -6,7 +6,7 @@ | @@ -6,7 +6,7 @@ | ||
| 6 | <div class="form-row"> | 6 | <div class="form-row"> |
| 7 | <div class="field-container"> | 7 | <div class="field-container"> |
| 8 | <%= f.label :label, class: 'control-label' %> | 8 | <%= f.label :label, class: 'control-label' %> |
| 9 | - <%= f.text_field :label, class: 'text-field form-control' %> | 9 | + <%= f.text_field :label, :required => true, class: 'text-field form-control' %> |
| 10 | </div> | 10 | </div> |
| 11 | <div class="help-container"> | 11 | <div class="help-container"> |
| 12 | <p> | 12 | <p> |
| @@ -22,7 +22,7 @@ | @@ -22,7 +22,7 @@ | ||
| 22 | <div class="field-container"> | 22 | <div class="field-container"> |
| 23 | <%= f.label :color, class: 'control-label' %> | 23 | <%= f.label :color, class: 'control-label' %> |
| 24 | <div class="input-group" id="color_div" style="z-index: 1010"> | 24 | <div class="input-group" id="color_div" style="z-index: 1010"> |
| 25 | - <%= f.text_field :color, class: 'form-control' %> | 25 | + <%= f.text_field :color, :required => true, class: 'form-control' %> |
| 26 | <span class="input-group-btn"> | 26 | <span class="input-group-btn"> |
| 27 | <input type="button" class="btn btn-info" id="pickcolor_button" value="Pick color" /> | 27 | <input type="button" class="btn btn-info" id="pickcolor_button" value="Pick color" /> |
| 28 | </span> | 28 | </span> |
| @@ -41,7 +41,7 @@ | @@ -41,7 +41,7 @@ | ||
| 41 | <div class="form-row"> | 41 | <div class="form-row"> |
| 42 | <div class="field-container"> | 42 | <div class="field-container"> |
| 43 | <%= f.label :grade, class: 'control-label' %> | 43 | <%= f.label :grade, class: 'control-label' %> |
| 44 | - <%= f.text_field :grade, class: 'text-field form-control' %> | 44 | + <%= f.text_field :grade, :required => true, class: 'text-field form-control' %> |
| 45 | </div> | 45 | </div> |
| 46 | <div class="help-container"> | 46 | <div class="help-container"> |
| 47 | <p> | 47 | <p> |
app/views/repositories/_form.html.erb
| @@ -7,7 +7,7 @@ | @@ -7,7 +7,7 @@ | ||
| 7 | <div class="form-row"> | 7 | <div class="form-row"> |
| 8 | <div class="field-container"> | 8 | <div class="field-container"> |
| 9 | <%= f.label :name, class: 'control-label' %> | 9 | <%= f.label :name, class: 'control-label' %> |
| 10 | - <%= f.text_field :name, class: 'text-field form-control' %> | 10 | + <%= f.text_field :name, :required => true, class: 'text-field form-control' %> |
| 11 | </div> | 11 | </div> |
| 12 | <div class="help-container"> | 12 | <div class="help-container"> |
| 13 | <p> | 13 | <p> |
| @@ -55,7 +55,7 @@ | @@ -55,7 +55,7 @@ | ||
| 55 | <div class="form-row"> | 55 | <div class="form-row"> |
| 56 | <div class="field-container"> | 56 | <div class="field-container"> |
| 57 | <%= f.label :address, class: 'control-label' %> | 57 | <%= f.label :address, class: 'control-label' %> |
| 58 | - <%= f.text_field :address, class: 'text-field form-control' %> | 58 | + <%= f.text_field :address, :required => true, class: 'text-field form-control' %> |
| 59 | </div> | 59 | </div> |
| 60 | <div class="help-container"> | 60 | <div class="help-container"> |
| 61 | <p> | 61 | <p> |