[Closes browsermedia#682] Improved :name input

<%= f.input :name, as: :name %> Now renders consistently formatted name field. * Removed:default option for text fields (documented alternative) * Show large full column input if no label. * Labels are disabled by default. * Placeholder is 'Name' by default.

Author: peakpg

app/helpers/cms/form_tag_helper.rb

@@ -16,6 +16,7 @@ def forecasting_a_new_section?(form_object)
  Cms::Section.with_path(form_object.object.class.path).first.nil?
  end
 
+
  def slug_source_if(boolean)
  if boolean
  {input_html: {class: 'slug-source'}}

app/inputs/cms_text_field_input.rb

@@ -0,0 +1,42 @@
+# A text field that is used for the Name fields of content.
+#
+# @example <%= f.name as: :name %>
+#
+# Add the following behaviors above and beyond
+# 1. Will generate a slug if the class requires it. (Requires a as: :path field to work)
+# 2. If no label is specified, it shows a larger than normal input which spans the full row.
+# 3. Labels are turned off by default.
+class NameInput < SimpleForm::Inputs::TextInput
+
+ def initialize(*args)
+ super(*args)
+ options[:label] = false if options[:label].nil?
+ options[:placeholder] = "Name" if options[:placeholder].nil?
+ end
+
+ def input
+ add_slug_source_for_content_that_needs_it
+
+ unless options[:label]
+ input_html_options[:class] << 'input-block-level input-xxlarge'
+ end
+
+ @builder.text_field(attribute_name, input_html_options).html_safe
+ end
+
+ protected
+
+ def add_slug_source_for_content_that_needs_it
+ if should_autogenerate_slug?
+ input_html_options[:class] << 'slug-source'
+ end
+ end
+
+ def should_autogenerate_slug?
+ content_requires_slug_field? && (object.new_record? || (object.name.blank? && object.slug.blank?))
+ end
+
+ def content_requires_slug_field?
+ object.class.requires_slug?
+ end
+end

app/inputs/name_input.rb

@@ -3,7 +3,7 @@
 <% end %>
 
 <h3>Form Settings</h3>
-<%= f.input :name, as: :cms_text_field %>
+<%= f.input :name, as: :name %>
 <%= f.input :slug, as: :path %>
 <%= f.input :description %>
 <%= hidden_field_tag "field_ids", @block.field_ids %>

app/views/cms/forms/_form.html.erb

@@ -5,7 +5,7 @@
  <%= f.hidden_field :lock_version %>
 <% end %>
 
-<%= f.input :name, label: false, placeholder: 'Page Name', input_html: {class: 'input-block-level input-xxlarge'} %>
+<%= f.input :name, placeholder: 'Page Name', as: :name %>
 <%= f.input :path, placeholder: 'Page Path' %>
 <%= f.input :template_file_name, collection: Cms::PageTemplate.options, label: "Template", as: :select, include_blank: false %>
 

app/views/cms/pages/_main_form.html.erb

@@ -1,5 +1,5 @@
 <%= f.input :name %>
 <%= f.input :sender, hint: "Enter the email for the 'from' field on any emails sent out." %>
-<%= f.input :subject, as: :cms_text_field, default: "A link you might be interested in" %>
+<%= f.input :subject, input_html: { value: f.object.subject || "A link you might be interested in" } %>
 <%= f.input :success_url, hint: "Leave blank to return the same page" %>
 <%= f.input :template, as: :template_editor %>

app/views/portlets/email_page/_form.html.erb

@@ -19,7 +19,7 @@
  </div>
  <% end %>
 <% end %>
-<%= f.input :name, placeholder: 'Name', label: false, input_html: {class: 'input-block-level input-xxlarge'} %>
+<%= f.input :name, as: :name %>
 <%= f.input :content_type,
  collection: Cms::ContentType.available,
  label_method: :display_name,

app/views/portlets/list/_form.html.erb

@@ -1,4 +1,4 @@
 <%= f.input :name %>
 <%= f.input :limit, :size => 4 %>
-<%= f.input :sizes, :default => TagCloudPortlet.default_sizes, as: :cms_text_field %>
+<%= f.input :sizes, input_html: { value: f.object.sizes || TagCloudPortlet.default_sizes } %>
 <%= f.input :template, as: :template_editor %>

app/views/portlets/tag_cloud/_form.html.erb

@@ -30,10 +30,10 @@ All of the existing simple_form input mappings are available. See http://simple-
 Here are some examples showing how to use the CMS specific inputs.
 
 ```
-<%= f.input :name, as: :cms_text_field, default: "Some Value" %>
+<%= f.input :name %>
 ```
 
-Generates the 'name' field, and a :path input if the content block is addressable. Can supply a default value that will be used when creating a new record.
+Generates the 'name' field as a text field.
 
 ```
 <%= f.input :expires_on, as: :date_picker %>

doc/features/simple_form_refactor.md

@@ -1,6 +1,7 @@
 # v4.0.0.beta
 
 * List Portlet [#678] - A convenient way to find content without custom coding.
+* NameInput [#682] - Improved :name input allows for consistent name fields look/feel. New content will be generated with it.
 
 ## [#678] List Portlet