chart title optional

CSC322-Grinnell · Dec 13, 2020 · f2daa8f · f2daa8f
1 parent 8c3fbe1
commit f2daa8f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/app/models/visualization.rb b/app/models/visualization.rb
@@ -6,9 +6,14 @@ class Visualization < ApplicationRecord
   validates :x_axis_title, presence: true
   validates :y_axis_title, presence: true
   validates :chart_type, presence: true
-  before_validation :x_axis_title_default, :y_axis_title_default
+  before_validation :chart_title_default, :x_axis_title_default, :y_axis_title_default
 
   private
+  def chart_title_default
+    if chart_title.empty?
+      self.chart_title = "Visualization"
+    end
+  end
   def x_axis_title_default
     if x_axis_title.empty?
       self.x_axis_title = "Variable"

diff --git a/app/views/visualizations/_form.html.erb b/app/views/visualizations/_form.html.erb
@@ -71,7 +71,7 @@
 
           <div class="chart-details form-row">
             <div class="viz-form-group">
-              <label for="chart_title">Chart Title <span style="color:red;">*</span></label>
+              <label for="chart_title">Chart Title</label>
               <%= f.text_field :chart_title, class: 'form-control' %>
             </div>