blacklight-spotlight 4.1.2 → 4.3.0

data/app/controllers/spotlight/roles_controller.rb

@@ -13,9 +13,9 @@ module Spotlight
       role = @exhibit.roles.build
       authorize! :edit, role
 
-      add_breadcrumb t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit
-      add_breadcrumb t(:'spotlight.configuration.sidebar.header'), exhibit_dashboard_path(@exhibit)
-      add_breadcrumb t(:'spotlight.configuration.sidebar.users'), exhibit_roles_path(@exhibit)
+      add_breadcrumb(t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit)
+      add_breadcrumb(t(:'spotlight.configuration.sidebar.header'), exhibit_dashboard_path(@exhibit))
+      add_breadcrumb(t(:'spotlight.configuration.sidebar.users'), exhibit_roles_path(@exhibit))
     end
 
     def update_all
@@ -24,7 +24,7 @@ module Spotlight
       if @exhibit.update(exhibit_params)
         notice = any_deleted ? t(:'helpers.submit.role.destroyed') : t(:'helpers.submit.role.updated')
         Spotlight::InviteUsersService.call(resource: @exhibit)
-        redirect_to exhibit_roles_path(@exhibit), notice: notice
+        redirect_to exhibit_roles_path(@exhibit), notice:
       else
         flash[:alert] = t(:'helpers.submit.role.batch_error', count: exhibit_params[:roles_attributes].to_unsafe_h.size)
         render action: 'index'

data/app/controllers/spotlight/search_configurations_controller.rb

@@ -19,9 +19,9 @@ module Spotlight
     end
 
     def edit
-      add_breadcrumb t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit
-      add_breadcrumb t(:'spotlight.configuration.sidebar.header'), exhibit_dashboard_path(@exhibit)
-      add_breadcrumb t(:'spotlight.configuration.sidebar.search_configuration'), edit_exhibit_search_configuration_path(@exhibit)
+      add_breadcrumb(t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit)
+      add_breadcrumb(t(:'spotlight.configuration.sidebar.header'), exhibit_dashboard_path(@exhibit))
+      add_breadcrumb(t(:'spotlight.configuration.sidebar.search_configuration'), edit_exhibit_search_configuration_path(@exhibit))
 
       @field_metadata = Spotlight::FieldMetadata.new(current_exhibit, blacklight_config.repository, @blacklight_configuration.blacklight_config)
     end

data/app/controllers/spotlight/searches_controller.rb

@@ -30,7 +30,7 @@ module Spotlight
 
     def edit
       @groups = @exhibit.groups
-      add_breadcrumb @search.full_title, edit_exhibit_search_path(@search.exhibit, @search)
+      add_breadcrumb(@search.full_title, edit_exhibit_search_path(@search.exhibit, @search))
       @exhibit = @search.exhibit
     end
 
@@ -78,7 +78,7 @@ module Spotlight
                else
                  t(:'helpers.submit.search.batch_error', model: Spotlight::Search.model_name.human.pluralize.downcase)
                end
-      redirect_back fallback_location: fallback_url, notice: notice
+      redirect_back fallback_location: fallback_url, notice:
     end
 
     protected
@@ -94,9 +94,9 @@ module Spotlight
 
     def attach_breadcrumbs
       e = @exhibit || @search&.exhibit
-      add_breadcrumb t(:'spotlight.exhibits.breadcrumb', title: e.title), e
-      add_breadcrumb t(:'spotlight.curation.sidebar.header'), exhibit_dashboard_path(e)
-      add_breadcrumb t(:'spotlight.curation.sidebar.browse'), exhibit_searches_path(e)
+      add_breadcrumb(t(:'spotlight.exhibits.breadcrumb', title: e.title), e)
+      add_breadcrumb(t(:'spotlight.curation.sidebar.header'), exhibit_dashboard_path(e))
+      add_breadcrumb(t(:'spotlight.curation.sidebar.browse'), exhibit_searches_path(e))
     end
 
     def batch_search_params

data/app/controllers/spotlight/sites_controller.rb

@@ -9,13 +9,13 @@ module Spotlight
     load_and_authorize_resource
 
     def edit
-      add_breadcrumb t(:'spotlight.sites.home'), root_url
-      add_breadcrumb t(:'spotlight.sites.edit.page_title')
+      add_breadcrumb(t(:'spotlight.sites.home'), root_url)
+      add_breadcrumb(t(:'spotlight.sites.edit.page_title'))
     end
 
     def edit_exhibits
-      add_breadcrumb t(:'spotlight.sites.home'), root_url
-      add_breadcrumb t(:'spotlight.sites.edit_exhibits.page_title')
+      add_breadcrumb(t(:'spotlight.sites.home'), root_url)
+      add_breadcrumb(t(:'spotlight.sites.edit_exhibits.page_title'))
     end
 
     def update

data/app/controllers/spotlight/tags_controller.rb

@@ -14,9 +14,9 @@ module Spotlight
 
     def index
       @tags = @exhibit.owned_tags
-      add_breadcrumb t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit
-      add_breadcrumb t(:'spotlight.curation.sidebar.header'), exhibit_dashboard_path(@exhibit)
-      add_breadcrumb t(:'spotlight.curation.sidebar.tags'), exhibit_tags_path(@exhibit)
+      add_breadcrumb(t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit)
+      add_breadcrumb(t(:'spotlight.curation.sidebar.header'), exhibit_dashboard_path(@exhibit))
+      add_breadcrumb(t(:'spotlight.curation.sidebar.tags'), exhibit_tags_path(@exhibit))
 
       respond_to do |format|
         format.html

data/app/controllers/spotlight/translations_controller.rb

@@ -21,7 +21,7 @@ module Spotlight
       if current_exhibit.update(exhibit_params)
         I18n.reload! # reload since we're memoizing
         notice = t(:'helpers.submit.spotlight_default.updated', model: current_exhibit.class.model_name.human.downcase)
-        redirect_to edit_exhibit_translations_path(current_exhibit, language: @language, tab: @tab), notice: notice
+        redirect_to edit_exhibit_translations_path(current_exhibit, language: @language, tab: @tab), notice:
       else
         render 'edit'
       end
@@ -31,7 +31,7 @@ module Spotlight
       if current_exhibit.update(import_exhibit_params)
         I18n.reload! # reload since we're memoizing
         notice = t(:'helpers.submit.spotlight_default.updated', model: current_exhibit.class.model_name.human.downcase)
-        redirect_to edit_exhibit_translations_path(current_exhibit, language: @language), notice: notice
+        redirect_to edit_exhibit_translations_path(current_exhibit, language: @language), notice:
       else
         render 'edit'
       end
@@ -40,9 +40,9 @@ module Spotlight
     private
 
     def attach_breadcrumbs
-      add_breadcrumb t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit
-      add_breadcrumb t(:'spotlight.curation.sidebar.header'), exhibit_dashboard_path(@exhibit)
-      add_breadcrumb t(:'spotlight.curation.sidebar.translations')
+      add_breadcrumb(t(:'spotlight.exhibits.breadcrumb', title: @exhibit.title), @exhibit)
+      add_breadcrumb(t(:'spotlight.curation.sidebar.header'), exhibit_dashboard_path(@exhibit))
+      add_breadcrumb(t(:'spotlight.curation.sidebar.translations'))
     end
 
     def exhibit_params

data/app/helpers/spotlight/application_helper.rb

@@ -37,7 +37,7 @@ module Spotlight
       initial_exception = nil
 
       ([self] + additional_locale_routing_scopes).each do |scope|
-        return scope.public_send(:url_for, params.to_unsafe_h.merge(locale: locale))
+        return scope.public_send(:url_for, params.to_unsafe_h.merge(locale:))
       rescue ActionController::UrlGenerationError => e
         initial_exception ||= e
       end
@@ -51,7 +51,7 @@ module Spotlight
 
     # Can search for named routes directly in the main app, omitting
     # the "main_app." prefix
-    def method_missing(method, *args, &block)
+    def method_missing(method, *args, &)
       if main_app_url_helper?(method)
         main_app.send(method, *args)
       else

data/app/helpers/spotlight/crop_helper.rb

@@ -11,7 +11,7 @@ module Spotlight
         concat f.hidden_field(:iiif_image_id)
         concat f.hidden_field(:iiif_region)
         concat f.hidden_field(:iiif_tilesource)
-        concat iiif_cropper_tag(f, initial_crop_selection: initial_crop_selection)
+        concat iiif_cropper_tag(f, initial_crop_selection:)
       end
     end
 

data/app/helpers/spotlight/pages_helper.rb

@@ -80,5 +80,12 @@ module Spotlight
     def configurations_for_current_page
       Spotlight::PageConfigurations.new(context: self, page: @page).as_json
     end
+
+    def resource_alt_text(options, default)
+      return '' if options[:decorative].present?
+      return options[:alt_text] if options[:alt_text].present?
+
+      default
+    end
   end
 end

data/app/helpers/spotlight/title_helper.rb

@@ -15,7 +15,7 @@ module Spotlight
     def page_title(section, title = nil)
       title ||= t(:'.header', default: '').presence
 
-      head_content = t(:'spotlight.html_admin_title', section: section, title: title) if section && title
+      head_content = t(:'spotlight.html_admin_title', section:, title:) if section && title
       head_content ||= section || title
       set_html_page_title(head_content)
 
@@ -28,7 +28,7 @@ module Spotlight
     end
 
     def set_html_page_title(title = nil)
-      @page_title = strip_tags(t(:'spotlight.html_title', title: title || t(:'.title', default: :'.header'), application_name: application_name)).html_safe
+      @page_title = strip_tags(t(:'spotlight.html_title', title: title || t(:'.title', default: :'.header'), application_name:)).html_safe
     end
   end
 end

data/app/javascript/spotlight/admin/blocks/browse_group_categories_block.js

@@ -82,7 +82,7 @@ SirTrevor.Blocks.BrowseGroupCategories = (function(){
     },
 
     item_options: function() { return `
-      '<label>
+      <label>
         <input type="hidden" name="display-item-counts" value="false" />
         <input type="checkbox" name="display-item-counts" value="true" checked />
         ${i18n.t("blocks:browse_group_categories:item_counts")}

data/app/javascript/spotlight/admin/blocks/resources_block.js

@@ -8,10 +8,24 @@ Core.Block.Resources = (function(){
     formable: true,
     autocompleteable: true,
     show_heading: true,
+    show_alt_text: true,
 
     title: function() { return i18n.t("blocks:" + this.type + ":title"); },
     description: function() { return i18n.t("blocks:" + this.type + ":description"); },
-
+    alt_text_guidelines: function() { 
+      if (this.show_alt_text) {
+        return i18n.t("blocks:alt_text_guidelines:intro"); 
+      }
+      return "";
+    },
+    alt_text_guidelines_link: function() {
+      if (this.show_alt_text) {
+        var link_url = i18n.t("blocks:alt_text_guidelines:link_url");
+        var link_label = i18n.t("blocks:alt_text_guidelines:link_label");
+        return '<a target="_blank" href="' + link_url + '">' +  link_label + '</a>'; 
+      }
+      return "";
+    },
     icon_name: "resources",
     blockGroup: function() { return i18n.t("blocks:group:items") },
 
@@ -21,6 +35,8 @@ Core.Block.Resources = (function(){
     show_secondary_field_key: "show-secondary-caption",
 
     display_checkbox: "display-checkbox",
+    decorative_checkbox: "decorative-checkbox",
+    alt_text_textarea: "alt-text-textarea",
 
     globalIndex: 0,
 
@@ -28,6 +44,13 @@ Core.Block.Resources = (function(){
       return [];
     },
 
+    _altTextFieldsHTML: function(index, data) {
+      if (this.show_alt_text) {
+        return this.altTextHTML(index, data);
+      }
+      return "";
+    },
+
     _itemPanel: function(data) {
       var index = "item_" + this.globalIndex++;
       var checked;
@@ -58,6 +81,7 @@ Core.Block.Resources = (function(){
                     <div class="main">
                       <div class="title card-title">${data.title}</div>
                       <div>${(data.slug || data.id)}</div>
+                      ${this._altTextFieldsHTML(index, data)}
                     </div>
                     <div class="remove float-right float-end">
                       <a data-item-grid-panel-remove="true" href="#">${i18n.t("blocks:resources:panel:remove")}</a>
@@ -94,6 +118,7 @@ Core.Block.Resources = (function(){
 
     createItemPanel: function(data) {
       var panel = this._itemPanel(data);
+      this.attachAltTextHandlers(panel);
       $(panel).appendTo($('.panels > ol', this.inner));
       $('[data-behavior="nestable"]', this.inner).trigger('change');
     },
@@ -126,11 +151,67 @@ Core.Block.Resources = (function(){
       return `<div class="form resources-admin clearfix">
         <div class="widget-header">
           ${this.description()}
+          ${this.alt_text_guidelines()}
+          ${this.alt_text_guidelines_link()}
         </div>
         ${this.content()}
       </div>`
     },
 
+    _altTextData: function(data) {
+      const isDecorative = data.decorative;
+      const altText = isDecorative ? '' : (data.alt_text || '');
+      const altTextBackup = data.alt_text_backup || '';
+      const placeholderAttr = isDecorative ? '' : `placeholder="${i18n.t("blocks:resources:alt_text:placeholder")}"`;
+      const disabledAttr = isDecorative ? 'disabled' : '';
+
+      return { isDecorative, altText, altTextBackup, placeholderAttr, disabledAttr };
+    },
+
+    altTextHTML: function(index, data) {
+      const { isDecorative, altText, altTextBackup, placeholderAttr, disabledAttr } = this._altTextData(data);
+      return `<div class="mt-2 pt-2 d-flex">
+          <div class="me-2 mr-2">
+            <label class="col-form-label pb-0 pt-1" for="${this.formId(this.alt_text_textarea + '_' + data.id)}">${i18n.t("blocks:resources:alt_text:alternative_text")}</label>
+            <div class="form-check mb-1 justify-content-end">
+              <input class="form-check-input" type="checkbox" 
+                id="${this.formId(this.decorative_checkbox + '_' + data.id)}" name="item[${index}][decorative]" ${isDecorative ? 'checked' : ''}>
+              <label class="form-check-label" for="${this.formId(this.decorative_checkbox + '_' + data.id)}">${i18n.t("blocks:resources:alt_text:decorative")}</label>
+            </div>
+          </div>
+          <div class="flex-grow-1 flex-fill d-flex">
+            <input type="hidden" name="item[${index}][alt_text_backup]" value="${altTextBackup}" />
+            <textarea class="form-control w-100" rows="2" ${placeholderAttr}
+              id="${this.formId(this.alt_text_textarea + '_' + data.id)}" name="item[${index}][alt_text]" ${disabledAttr}>${altText}</textarea>
+          </div>
+        </div>`
+    },
+
+    attachAltTextHandlers: function(panel) {
+      if (this.show_alt_text) {
+        const decorativeCheckbox = $('input[name$="[decorative]"]', panel);
+        const altTextInput = $('textarea[name$="[alt_text]"]', panel);
+        const altTextBackupInput = $('input[name$="[alt_text_backup]"]', panel);
+
+        decorativeCheckbox.on('change', function() {
+          const isDecorative = this.checked;
+          if (isDecorative) {
+            altTextBackupInput.val(altTextInput.val());
+            altTextInput.val('');
+          } else {
+            altTextInput.val(altTextBackupInput.val());
+          }
+          altTextInput
+            .prop('disabled', isDecorative)
+            .attr('placeholder', isDecorative ? '' : i18n.t("blocks:resources:alt_text:placeholder"));
+        });
+
+        altTextInput.on('input', function() {
+          $(this).data('lastValue', $(this).val());
+        });
+      }
+    },
+
     onBlockRender: function() {
       SpotlightNestable.init($('[data-behavior="nestable"]', this.inner));
 

data/app/javascript/spotlight/admin/blocks/solr_documents_embed_block.js

@@ -4,7 +4,7 @@ SirTrevor.Blocks.SolrDocumentsEmbed = (function(){
 
   return SirTrevor.Blocks.SolrDocumentsBase.extend({
     type: "solr_documents_embed",
-
+    show_alt_text: false,
     icon_name: "item_embed",
 
     item_options: function() { return "" },

data/app/javascript/spotlight/admin/blocks/uploaded_items_block.js

@@ -96,6 +96,7 @@ SirTrevor.Blocks.UploadedItems = (function(){
                     <label for="${this.formId('link_' + dataId)}" class="col-form-label col-md-3">${i18n.t("blocks:uploaded_items:link")}</label>
                     <input type="text" class="form-control col" id="${this.formId('link_' + dataId)}" name="item[${index}][link]" data-field="link"/>
                   </div>
+                  ${this._altTextFieldsHTML(index, data)}
                 </div>
                 <div class="remove float-right float-end">
                   <a data-item-grid-panel-remove="true" href="#">${i18n.t("blocks:resources:panel:remove")}</a>
@@ -123,6 +124,8 @@ SirTrevor.Blocks.UploadedItems = (function(){
       return `<div class="form oembed-text-admin clearfix">
         <div class="widget-header">
           ${this.description()}
+          ${this.alt_text_guidelines()}
+          ${this.alt_text_guidelines_link()}
         </div>
         <div class="row">
           <div class="form-group mb-3 col-md-8">
@@ -142,6 +145,24 @@ SirTrevor.Blocks.UploadedItems = (function(){
       </div>`
     },
 
+    altTextHTML: function(index, data) {
+      const { isDecorative, altText, altTextBackup, placeholderAttr, disabledAttr } = this._altTextData(data);
+      return `
+      <div class="field row mr-3 me-3">
+        <div class="col-lg-3 ps-md-2 pl-md-2">
+          <label class="col-form-label text-nowrap pb-0 pt-1 justify-content-md-start justify-content-lg-end d-flex" for="${this.formId(this.alt_text_textarea + '_' + data.id)}">${i18n.t("blocks:resources:alt_text:alternative_text")}</label>
+          <div class="form-check d-flex justify-content-md-start justify-content-lg-end">
+            <input class="form-check-input" type="checkbox" 
+              id="${this.formId(this.decorative_checkbox + '_' + data.id)}" name="item[${index}][decorative]" ${isDecorative ? 'checked' : ''}>
+            <label class="form-check-label" for="${this.formId(this.decorative_checkbox + '_' + data.id)}">${i18n.t("blocks:resources:alt_text:decorative")}</label>
+          </div>
+        </div>
+        <input type="hidden" name="item[${index}][alt_text_backup]" value="${altTextBackup}" />
+        <textarea class="col-lg-9" rows="2" ${placeholderAttr}
+          id="${this.formId(this.alt_text_textarea + '_' + data.id)}" name="item[${index}][alt_text]" ${disabledAttr}>${altText}</textarea>
+      </div>`
+    },
+
     zpr_key: 'zpr_link'
   });
 })();

data/app/javascript/spotlight/admin/edit_in_place.js

@@ -43,5 +43,32 @@ export default class {
         return false;
       });
     })
+
+    $("[data-behavior='restore-default']").each(function(){
+      var hidden = $("[data-default-value]", $(this));
+      var value = $($("[data-in-place-edit-target]", $(this)).data('in-place-edit-target'), $(this));
+      var button = $("[data-restore-default]", $(this));
+
+      hidden.on('keypress', function(e) {
+        if(e.which == 13) {
+          hidden.trigger('blur');
+          return false;
+        }
+      });
+
+      hidden.on('blur', function(){
+        if( $(this).val() == $(this).data('default-value') ) {
+          button.addClass('d-none');
+        } else {
+          button.removeClass('d-none');
+        }
+      });
+      button.on('click', function(e){
+        e.preventDefault();
+        hidden.val(hidden.data('default-value'));
+        value.text(hidden.data('default-value'));
+        button.hide();
+      });
+    });
   }
 }

data/app/javascript/spotlight/admin/index.js

@@ -8,7 +8,6 @@ import 'Path.Drag'
 
 import AddAnother from 'spotlight/admin/add_another'
 import AddNewButton from 'spotlight/admin/add_new_button'
-import Appearance from 'spotlight/admin/appearance'
 import BlacklightConfiguration from 'spotlight/admin/blacklight_configuration'
 import CopyEmailAddress from 'spotlight/admin/copy_email_addresses'
 import Croppable from 'spotlight/admin/croppable'
@@ -60,7 +59,6 @@ export default class {
   connect() {
     new AddAnother().connect()
     new AddNewButton().connect()
-    new Appearance().connect()
     new CopyEmailAddress().connect()
     new Croppable().connect()
     new EditInPlace().connect()

data/app/javascript/spotlight/admin/sir-trevor/locales.js

@@ -51,6 +51,11 @@ SirTrevor.Locales.en.blocks = $.extend(SirTrevor.Locales.en.blocks, {
       drag: "Drag",
       display: "Display?",
       remove: "Remove"
+    },
+    alt_text: {
+      decorative: "Decorative",
+      alternative_text: "Alternative text",
+      placeholder: "Enter alt text for this item..."
     }
   },
 
@@ -116,5 +121,11 @@ SirTrevor.Locales.en.blocks = $.extend(SirTrevor.Locales.en.blocks, {
   group: {
     undefined: "Standard widgets",
     items: "Exhibit item widgets"
+  },
+
+  alt_text_guidelines: {
+    intro: 'For each item, please enter alternative text or appropriately check the decorative box. ',
+    link_label: 'Guidelines for writing alt text.',
+    link_url: 'https://www.w3.org/WAI/tutorials/images/' 
   }
 });

data/app/jobs/concerns/spotlight/job_tracking.rb

@@ -55,7 +55,7 @@ module Spotlight
     private
 
     def find_or_initialize_job_tracker
-      JobTracker.find_or_create_by(job_id: job_id) do |tracker|
+      JobTracker.find_or_create_by(job_id:) do |tracker|
         tracker.job_class = self.class.name
         tracker.status = 'enqueued'
       end

data/app/jobs/spotlight/add_tags_job.rb

@@ -19,12 +19,12 @@ module Spotlight
         document.reindex(update_params: {})
         progress&.increment
       rescue StandardError => e
-        job_tracker.append_log_entry(type: :error, exhibit: exhibit, message: e.to_s)
+        job_tracker.append_log_entry(type: :error, exhibit:, message: e.to_s)
         @errors += 1
         mark_job_as_failed!
       end
       exhibit.blacklight_config.repository.connection.commit
-      job_tracker.append_log_entry(type: :info, exhibit: exhibit, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
+      job_tracker.append_log_entry(type: :info, exhibit:, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
     end
     # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
   end

data/app/jobs/spotlight/add_uploads_from_csv.rb

@@ -44,7 +44,7 @@ module Spotlight
 
         resource = Spotlight::Resources::Upload.new(
           data: row,
-          exhibit: exhibit
+          exhibit:
         )
         resource.build_upload(remote_image_url: url) unless url == '~'
 

data/app/jobs/spotlight/change_visibility_job.rb

@@ -21,12 +21,12 @@ module Spotlight
         document.reindex(update_params: {})
         progress&.increment
       rescue StandardError => e
-        job_tracker.append_log_entry(type: :error, exhibit: exhibit, message: e.to_s)
+        job_tracker.append_log_entry(type: :error, exhibit:, message: e.to_s)
         @errors += 1
         mark_job_as_failed!
       end
       exhibit.blacklight_config.repository.connection.commit
-      job_tracker.append_log_entry(type: :info, exhibit: exhibit, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
+      job_tracker.append_log_entry(type: :info, exhibit:, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
     end
     # rubocop:enable Metrics/MethodLength
   end

data/app/jobs/spotlight/process_bulk_updates_csv_job.rb

@@ -20,13 +20,13 @@ module Spotlight
           process_row(exhibit, row)
           progress&.increment
         rescue StandardError => e
-          job_tracker.append_log_entry(type: :error, exhibit: exhibit, message: e.to_s)
+          job_tracker.append_log_entry(type: :error, exhibit:, message: e.to_s)
           errors += 1
           mark_job_as_failed!
         end
 
         exhibit.blacklight_config.repository.connection.commit
-        job_tracker.append_log_entry(type: :info, exhibit: exhibit, message: "#{progress&.progress} of #{progress&.total} (#{errors} errors)")
+        job_tracker.append_log_entry(type: :info, exhibit:, message: "#{progress&.progress} of #{progress&.total} (#{errors} errors)")
       end
     end
     # rubocop:enable Metrics/AbcSize, Metrics/MethodLength

data/app/jobs/spotlight/reindex_exhibit_job.rb

@@ -30,7 +30,7 @@ module Spotlight
 
     def perform_later_in_batches(exhibit, of:)
       last = 0
-      exhibit.resources.select(:id).in_batches(of: of) do |batch|
+      exhibit.resources.select(:id).in_batches(of:) do |batch|
         last = batch.last.id
         Spotlight::ReindexJob.perform_later(exhibit, reports_on: job_tracker, start: batch.first.id, finish: batch.last.id)
       end

data/app/jobs/spotlight/reindex_job.rb

@@ -29,7 +29,7 @@ module Spotlight
       error_handler = lambda do |pipeline, exception, _data|
         job_tracker.append_log_entry(
           type: :error,
-          exhibit: exhibit,
+          exhibit:,
           message: exception.to_s,
           backtrace: exception.backtrace.first(5).join("\n"),
           resource_id: (pipeline.source.id if pipeline.source.respond_to?(:id))
@@ -38,8 +38,8 @@ module Spotlight
         errors += 1
       end
 
-      resource_list(exhibit_or_resources, start: start, finish: finish).each do |resource|
-        resource.reindex(touch: false, commit: false, job_tracker: job_tracker, additional_data: job_data, on_error: error_handler) do |*|
+      resource_list(exhibit_or_resources, start:, finish:).each do |resource|
+        resource.reindex(touch: false, commit: false, job_tracker:, additional_data: job_data, on_error: error_handler) do |*|
           progress&.increment
         end
       rescue StandardError => e
@@ -48,14 +48,14 @@ module Spotlight
 
       job_tracker.append_log_entry(
         type: :summary,
-        exhibit: exhibit,
+        exhibit:,
         message: I18n.t(
           'spotlight.job_trackers.show.messages.status.in_progress',
           progress: progress.progress,
           total: progress.total,
           errors: (I18n.t('spotlight.job_trackers.show.messages.errors', count: errors) if errors.positive?)
         ),
-        progress: progress.progress, total: progress.total, errors: errors
+        progress: progress.progress, total: progress.total, errors:
       )
     end
     # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
@@ -85,7 +85,7 @@ module Spotlight
 
     def resource_list(exhibit_or_resources, start: nil, finish: nil)
       if exhibit_or_resources.is_a?(Spotlight::Exhibit)
-        exhibit_or_resources.resources.find_each(start: start, finish: finish)
+        exhibit_or_resources.resources.find_each(start:, finish:)
       else
         Array(exhibit_or_resources)
       end

data/app/jobs/spotlight/remove_tags_job.rb

@@ -19,12 +19,12 @@ module Spotlight
         document.reindex(update_params: {})
         progress&.increment
       rescue StandardError => e
-        job_tracker.append_log_entry(type: :error, exhibit: exhibit, message: e.to_s)
+        job_tracker.append_log_entry(type: :error, exhibit:, message: e.to_s)
         @errors += 1
         mark_job_as_failed!
       end
       exhibit.blacklight_config.repository.connection.commit
-      job_tracker.append_log_entry(type: :info, exhibit: exhibit, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
+      job_tracker.append_log_entry(type: :info, exhibit:, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
     end
     # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
   end

data/app/jobs/spotlight/rename_tags_job.rb

@@ -20,13 +20,13 @@ module Spotlight
         exhibit.tag(sidecar, with: all_tags, on: :tags)
         sidecar.document.reindex(update_params: {})
       rescue StandardError => e
-        job_tracker.append_log_entry(type: :error, exhibit: exhibit, message: e.to_s)
+        job_tracker.append_log_entry(type: :error, exhibit:, message: e.to_s)
         @errors += 1
         mark_job_as_failed!
       end
 
       exhibit.blacklight_config.repository.connection.commit
-      job_tracker.append_log_entry(type: :info, exhibit: exhibit, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
+      job_tracker.append_log_entry(type: :info, exhibit:, message: "#{progress.progress} of #{progress.total} (#{@errors} errors)")
     end
     # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
   end

data/app/models/breadcrumb.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+# Breadcrumb model
+class Breadcrumb
+  attr_reader :name, :path
+
+  def initialize(name, path)
+    @name = name
+    @path = path
+  end
+
+  def link?
+    @path.present?
+  end
+end

data/app/models/concerns/spotlight/exhibit_defaults.rb

@@ -35,7 +35,7 @@ module Spotlight
 
     def initialize_main_navigation
       default_main_navigations.each_with_index do |nav_type, weight|
-        main_navigations.create nav_type: nav_type, weight: weight
+        main_navigations.create nav_type:, weight:
       end
     end