BL-1171: Update blacklight_range_limit.

* Updates blacklight_range_limit to latest version. * Patches JS issue. * Hides remove button (to keep current behavior)
tulibraries · Jun 23, 2020 · 4e2c96f · 4e2c96f
1 parent d7a01f9
commit 4e2c96f
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 10 deletions.
diff --git a/Gemfile b/Gemfile
@@ -56,7 +56,7 @@ gem "tzinfo-data", platforms: [:mingw, :mswin, :x64_mingw, :jruby]
 gem "blacklight", "~> 7.8.0"
 gem "blacklight_advanced_search", git: "https://github.com/projectblacklight/blacklight_advanced_search.git"
 gem "blacklight-marc", git: "https://github.com/projectblacklight/blacklight-marc.git", ref: "v7.0.0.rc1"
-gem "blacklight_range_limit", git: "https://github.com/projectblacklight/blacklight_range_limit.git", ref: "v7.0.0.rc2"
+gem "blacklight_range_limit", git: "https://github.com/projectblacklight/blacklight_range_limit.git", ref: "v7.8.0"
 group :development, :test do
   gem "solr_wrapper", ">= 0.3"
   gem "rspec-rails"

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -21,14 +21,11 @@ GIT
 
 GIT
   remote: https://github.com/projectblacklight/blacklight_range_limit.git
-  revision: 6826dfb8b73554a79f26ca7ff375b0062e37bb22
-  ref: v7.0.0.rc2
+  revision: 44bfecba9d3c123fa01d6874288b0dc452fdb2a6
+  ref: v7.8.0
   specs:
-    blacklight_range_limit (7.0.0.rc2)
-      blacklight
-      jquery-rails
-      rails (>= 3.0)
-      tether-rails
+    blacklight_range_limit (7.8.0)
+      blacklight (>= 7.0)
 
 GIT
   remote: https://github.com/tulibraries/alma_rb.git
@@ -506,8 +503,6 @@ GEM
       json
     term-ansicolor (1.7.1)
       tins (~> 1.0)
-    tether-rails (1.4.0)
-      rails (>= 3.1)
     thor (1.0.1)
     thread_safe (0.3.6)
     tilt (2.0.10)

diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js
@@ -200,3 +200,9 @@ $(document).on('turbolinks:load', function() {
 		document.getElementById("main-toggler-icon").classList.toggle("change");
 	}
 }
+
+// This is hack to fix race condition bug in blacklight_range_limit gem.
+// REF: BL-1171 and project_blacklight/blacklight_range_limit#111
+window.addEventListener('load', function(event) {
+   $("#facet-pub_date_sort").trigger("shown.bs.collapse");
+});
diff --git a/app/assets/stylesheets/icons.css.erb b/app/assets/stylesheets/icons.css.erb
@@ -279,6 +279,10 @@ span.remove-icon {
   padding-left: 1.25rem;
 }
 
+.range_limit span.remove-icon {
+  display: none;
+}
+
 span.plus-icon {
   background: transparent url(<%= asset_path "icons/plus.svg" %>) no-repeat top left;
   padding-left: 1.25rem;

diff --git a/app/assets/stylesheets/tucob.scss.erb b/app/assets/stylesheets/tucob.scss.erb
@@ -243,6 +243,12 @@ img {
   to { -webkit-transform: rotate(360deg); }
 }
 
+// Overrides blacklight_range_limit settings.
+.range_limit .chart_js {
+  margin-left: 0px !important;
+  margin-right: 0px !important;
+}
+
 // This is a trick that can be used when chrome tools is displaying extra white space
 // It helps to identify which element is causing the problem
 // * {