Merge pull request #3732 from projectblacklight/doc-id

Push the default id for the show page back to the partial.

Author: jcoyne

app/components/blacklight/document_component.rb

@@ -93,7 +93,7 @@ def initialize(document: nil, partials: nil,
 
       @component = component
       @title_component = title_component
-      @id = id || ('document' if show)
+      @id = id
       @classes = classes
 
       @counter = counter

app/views/catalog/_show_main_content.html.erb

@@ -3,7 +3,7 @@
 <% content_for(:head) { render_link_rel_alternates } %>
 
 <% document_component = blacklight_config.view_config(:show).document_component -%>
-<%= render document_component.new(document: document_presenter(@document), component: :div, show: true, partials: blacklight_config.view_config(:show).partials) do |component| %>
+<%= render document_component.new(id: 'document', document: document_presenter(@document), component: :div, show: true, partials: blacklight_config.view_config(:show).partials) do |component| %>
   <% component.with_title(as: 'h1', classes: '', link_to_document: false, actions: false) %>
   <% component.with_footer do %>
     <% if @document.respond_to?(:export_as_openurl_ctx_kev) %>

spec/components/blacklight/document_component_spec.rb

@@ -136,13 +136,6 @@
       vc_test_controller.action_name = "show"
     end
 
-    it 'renders with an id' do
-      component.with_body { '-' }
-      render_inline component
-
-      expect(page).to have_css 'article#document'
-    end
-
     it 'renders a title' do
       render_inline component