From 80f9855f03c639307037a72d342c500806831d91 Mon Sep 17 00:00:00 2001 From: Sean Aery Date: Thu, 13 Jun 2024 14:06:19 -0400 Subject: [PATCH] Use main blacklight-hierarchy gem instead of TRLN's local fork. Resolves TD-1329. - TRLN's enhancements are now in blacklight-hierarchy as of 6.3.0 - Adds ability for TRLN partners to customize expand/collapse SVG icon & color via CSS --- VERSION | 2 +- .../trln_argon/trln_argon_dependencies.scss | 2 + .../trln_argon/trln_argon_shared.scss | 53 +++++++------------ .../trln_argon_variables_defaults.scss | 1 - .../trln_argon/install_generator.rb | 13 ----- lib/trln_argon/controller_override.rb | 5 +- lib/trln_argon/engine.rb | 1 + trln_argon.gemspec | 3 +- 8 files changed, 27 insertions(+), 53 deletions(-) diff --git a/VERSION b/VERSION index 2bf1c1cc..f90b1afc 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -2.3.1 +2.3.2 diff --git a/app/assets/stylesheets/trln_argon/trln_argon_dependencies.scss b/app/assets/stylesheets/trln_argon/trln_argon_dependencies.scss index e7b3dd57..41f85137 100644 --- a/app/assets/stylesheets/trln_argon/trln_argon_dependencies.scss +++ b/app/assets/stylesheets/trln_argon/trln_argon_dependencies.scss @@ -4,6 +4,8 @@ @import 'blacklight_advanced_search'; +@import 'blacklight/hierarchy/hierarchy'; + @import 'font-awesome'; @import 'chosen'; diff --git a/app/assets/stylesheets/trln_argon/trln_argon_shared.scss b/app/assets/stylesheets/trln_argon/trln_argon_shared.scss index 2ad12237..88942605 100644 --- a/app/assets/stylesheets/trln_argon/trln_argon_shared.scss +++ b/app/assets/stylesheets/trln_argon/trln_argon_shared.scss @@ -554,18 +554,28 @@ body.blacklight-trln-show { /* FACETS */ /* =============== */ -$hierarchy-facet-expand-icon: url("data:image/svg+xml,"); +// Blacklight Hierarchy Icons +// https://github.com/sul-dlss/blacklight-hierarchy/blob/main/README.md#changing-the-icons + +$b-h-closed-icon: url("data:image/svg+xml,"); + +$b-h-opened-icon: url("data:image/svg+xml,"); .facet-hierarchy { - ul { - padding-left: 0.5rem; - li { - margin-top: 0.25rem; - margin-bottom: 0.25rem; - &.h-leaf { - // Keep this padding-left the same width as .toggle-handle - padding-left: 1.2rem; - } + .twiddle > .toggle-handle { + .toggle-icon { + background-color: $gray-600; + min-width: 1rem; + min-height: 1rem; + background-image: none; + mask-repeat: no-repeat; + mask-position: center; + } + .closed { + mask-image: $b-h-closed-icon; + } + .opened { + mask-image: $b-h-opened-icon; } } @@ -573,28 +583,6 @@ $hierarchy-facet-expand-icon: url("data:image/svg+xml, [['f'], ':'] - # TRLN CUSTOMIZATION adds an optional third element to specify - # a custom FacetItemPresenter. + # An optional third element can specify a custom FacetItemPresenter. cnf_components[0..-2].join('_') => [[cnf_components[-1]], '|'], lf_components[0..-2].join('_') => [[lf_components[-1]], diff --git a/lib/trln_argon/engine.rb b/lib/trln_argon/engine.rb index 61111ba1..86829b85 100644 --- a/lib/trln_argon/engine.rb +++ b/lib/trln_argon/engine.rb @@ -1,5 +1,6 @@ require 'blacklight' require 'blacklight_advanced_search' +require 'blacklight-hierarchy' require 'blacklight_range_limit' require 'rails_autolink' require 'library_stdnums' diff --git a/trln_argon.gemspec b/trln_argon.gemspec index 235101f0..db3035e4 100644 --- a/trln_argon.gemspec +++ b/trln_argon.gemspec @@ -21,8 +21,7 @@ Gem::Specification.new do |s| s.add_dependency 'rails', '~> 7' s.add_dependency 'blacklight', '~> 7.0' s.add_dependency 'blacklight_advanced_search', '~> 8.0.0.alpha2' - # TODO: when trln-blacklight-hierarchy fork is published on rubygems, reference it here - # s.add_dependency 'trln-blacklight-hierarchy' + s.add_dependency 'blacklight-hierarchy', '~> 6.3' s.add_dependency 'blacklight_range_limit', '~> 7' s.add_dependency 'git', '>= 1.11.0', "< 2" s.add_dependency 'rails_autolink', '~> 1.1'