diff --git a/app/assets/stylesheets/sufia/_collections.scss b/app/assets/stylesheets/sufia/_collections.scss index e951c85348..ce145aaa08 100644 --- a/app/assets/stylesheets/sufia/_collections.scss +++ b/app/assets/stylesheets/sufia/_collections.scss @@ -82,3 +82,7 @@ form.per_page { .presenter-title { padding-left: 20px; } + +.required { + color: $required-color; +} diff --git a/app/assets/stylesheets/sufia/_settings.scss b/app/assets/stylesheets/sufia/_settings.scss index 97324480bb..926b807eb1 100644 --- a/app/assets/stylesheets/sufia/_settings.scss +++ b/app/assets/stylesheets/sufia/_settings.scss @@ -18,8 +18,9 @@ $icon-background-color-hover: $classic-white !default; $icon-border-color-hover: $gray-base !default; $icon-font-color-hover: $gray-base !default; -// Collection Icon +// Collection $collection-icon-search-lg: 8em !default; +$required-color: $vermilion !default; // File Show Page $file-show-term-color: $gray-dark !default; diff --git a/app/forms/sufia/forms/collection_form.rb b/app/forms/sufia/forms/collection_form.rb index 909fb57ac1..26af367cc3 100644 --- a/app/forms/sufia/forms/collection_form.rb +++ b/app/forms/sufia/forms/collection_form.rb @@ -1,8 +1,14 @@ module Sufia::Forms class CollectionForm < CurationConcerns::Forms::CollectionEditForm + self.required_fields = [:title] + + # Fields that are required for Collections + def primary_terms + required_fields + end + def rendered_terms - [:title, - :creator, + [:creator, :contributor, :description, :keyword, diff --git a/app/views/collections/_form.html.erb b/app/views/collections/_form.html.erb index e91becbd6e..c726709983 100644 --- a/app/views/collections/_form.html.erb +++ b/app/views/collections/_form.html.erb @@ -1,10 +1,16 @@ <%= simple_form_for @form, html: { class: 'editor' } do |f| %>
-

Descriptions * indicates required fields

- <% f.object.rendered_terms.each do |term| %> - <%= render_edit_field_partial term, f: f %> - <% end %> +

Descriptions

+

* indicates required fields

+
+ <% f.object.primary_terms.each do |term| %> + <%= render_edit_field_partial(term, f: f) %> + <% end %> + <% f.object.rendered_terms.each do |term| %> + <%= render_edit_field_partial term, f: f %> + <% end %> +
<%= hidden_field_tag :type, params[:type] %> <% if params[:batch_document_ids].present? %> <% params[:batch_document_ids].each do |batch_item| %>