Skip to content

Add contents hierarchy to collection display (ARC-149) - #67

Draft
ssciolla wants to merge 17 commits into
mainfrom
arc-149-contents
Draft

Add contents hierarchy to collection display (ARC-149)#67
ssciolla wants to merge 17 commits into
mainfrom
arc-149-contents

Conversation

@ssciolla

@ssciolla ssciolla commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

To Do

  • Add Contents to sidebar navigation
  • Add Aeonform functionality - see inventory of Aeon code in umich, especially aeonform.js https://mlit.atlassian.net/browse/ARC-161
  • Fix navbar styling
  • Investigate wrapping of Javascript (IIFE) (can we remove it?) Yes. With current modules method not necessary, but good to have for standalone js.
  • Finalize JS loading pipeline
  • Fix strong parameters? (Determined no action was needed. The log message is originating from the below line and just is designed to filter out unneeded parameters before it goes to the search_service. The log message will go away in production.)
    Unpermitted parameters: :contents, :key, :paginate. Context: { controller: CatalogController, action: hierarchy, request: #<ActionDispatch::Request:0x0000000126c232c0>, params: {"contents" => "true", "key" => "-document", "paginate" => "true", "controller" => "catalog", "action" => "hierarchy", "id" => "umich-bhl-032"} }
    
  • Tidy up show more/ show less (truncation control shows even when it's not needed).
  • Fix indentation
  • Make interesting(?) items links (items are linked always right now, no counter is in use)
    According to the Claude analysis is_interesting items are linked. Counter is not enabled on label-only items.
    • What's the deal with counters?
      Blacklight uses the counter option in link_to_document for search session tracking. When it's supplied:
      - The generated link includes tracking data with the counter value
      - Clicking through to a record lets Blacklight record which position in the results the user clicked.
      - It powers features on the show page like "« Previous | 123 of 456 | Next »" navigation
  • Make sure box info is showing up in Contents hierarchy

Comment thread app/javascript/aeonform.js
@ssciolla ssciolla added the enhancement New feature or request label Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants