blacklight-gallery 2.0.1 → 3.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 65b060ce585a9a48514531e9b45d9a7dfa27da2afe2dc6590843f554a5b636ae
-  data.tar.gz: 8563b48e338ee0c1f7dc8f76786da529d0503356e57256fd0e5c1af1d561be79
+  metadata.gz: 3139dbdcecabc780a4bdb03f4477325a69e289ca323e1d0cb10ea611da87ae58
+  data.tar.gz: 919879da25125a70cd46bb6343c04be75388ea9faa464ad53dfed32dc678775f
 SHA512:
-  metadata.gz: d91c6ab608729eceda638985b87b44184e74af0992b064d5a354bb3a964873f13f1ba3bb37f22d91942ec7e2316841bdc6460a9a203f1e7df0e697d72af4b3f7
-  data.tar.gz: bac396d723403aeef4c2e4fc5d0757a5a2ce77b89fd84dd0d67da699add3ec1f1e84d34c4912542e1853167d13df07a867b17fef14c3325fc94801ba923948fa
+  metadata.gz: bd79b771b4ffd243ebf25eac0c97693695fc86a8ec72d8eacce24929095a8e135808dd5ce0ff5a25c6b72cf51c89cd89579ede20323d21dab7b857a119360bf0
+  data.tar.gz: a667e4bed21e3e92736db8a8e607cb7367bc3d5021a6af38d14159cdbf562d3f0eef9d0b4555ba0e464afc3c999bc2a36e894ce353e9bf411f2230b2666f6df1

data/.github/workflows/ruby.yml

@@ -0,0 +1,45 @@
+name: CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        rails_version: [6.1.1]
+        ruby: [2.7, 3.0]
+    env:
+      RAILS_VERSION: ${{ matrix.rails_version }}
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby ${{ matrix.ruby }}
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+    - name: Install dependencies with Rails ${{ matrix.rails_version }}
+      run: bundle install
+    - name: Run tests
+      run: bundle exec rake
+  test_rails52:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        rails_version: [5.2.4.4, 6.0.3.4]
+        ruby: [2.7]
+    env:
+      RAILS_VERSION: ${{ matrix.rails_version }}
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby ${{ matrix.ruby }}
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+    - name: Install dependencies with Rails ${{ matrix.rails_version }}
+      run: bundle install
+    - name: Run tests
+      run: bundle exec rake

data/README.md

@@ -1,5 +1,5 @@
 # Blacklight::Gallery
-[![Gem Version](https://badge.fury.io/rb/blacklight-gallery.svg)](http://badge.fury.io/rb/blacklight-gallery)
+[![Gem Version](https://badge.fury.io/rb/blacklight-gallery.svg)](http://badge.fury.io/rb/blacklight-gallery)  [![Build Status](https://travis-ci.org/projectblacklight/blacklight-gallery.svg?branch=master)](https://travis-ci.org/projectblacklight/blacklight-gallery)
 
 Gallery views for Blacklight search results
 

data/app/assets/javascripts/blacklight_gallery/default.js

@@ -1,5 +1,4 @@
 //= require imagesloaded.min.js
 //= require masonry.min.js
 //= require blacklight_gallery/slideshow
-//= require blacklight_gallery/osd_viewer
 //= require blacklight_gallery/masonry

data/app/assets/javascripts/blacklight_gallery/masonry.js

@@ -10,5 +10,5 @@
 })(jQuery);
 
 Blacklight.onLoad(function() {
-  $('[data-behavior="masonry-gallery"]').BlacklightMasonry();
+  $('.documents-masonry').BlacklightMasonry();
 });

data/app/assets/javascripts/blacklight_gallery/osd_viewer.js

@@ -1 +1 @@
-//= require openseadragon/rails
+//= require openseadragon/rails

data/app/assets/javascripts/blacklight_gallery/slideshow.js

@@ -85,7 +85,6 @@
       });
 
       $(document).on('click', '[data-slide], [data-slide-to]', function(e) {
-
         e.preventDefault();
 
         pos = parseInt($(this).attr('data-slide-to'), 10) || $(this).attr('data-slide');
@@ -122,5 +121,5 @@
 
 
 Blacklight.onLoad(function() {
-  $('#slideshow').slideshow();
+  $('.documents-slideshow').slideshow();
 });

data/app/assets/stylesheets/blacklight_gallery/_gallery.scss

@@ -1,21 +1,7 @@
-.gallery {
-
-  display: -webkit-box;
-  display: -moz-box;
-  display: -ms-flexbox;
-  display: -webkit-flex;
-  display: flex;
-  flex-flow: row wrap;
-  -webkit-flex-wrap: wrap;
-
-
+.documents-gallery {
   .document {
     border-bottom: none;
-
-    -webkit-box-flex: 1;
-    -moz-box-flex: 1;
-    -ms-flex: 1;
-    flex: 1;
+    display: flex;
 
     min-height: 250px;
     -webkit-flex: 1 0 250px;
@@ -39,16 +25,20 @@
 
   .document-metadata {
     dt, dd {
+      flex: 0 0 100%;
+      max-width: 100%;
+      padding-left: 0;
+      padding-right: 0;
       float: none;
       width: auto;
       clear: none;
       text-align: left;
       margin: 0;
     }
-
   }
 }
 
-#documents .gallery, .masonry-gallery, .slideshow-documents {
+.documents-gallery, .documents-masonry, .documents-slideshow {
+  padding-bottom: $spacer;
   border-bottom: $pagination-border-width solid $pagination-border-color
 }

data/app/assets/stylesheets/blacklight_gallery/_masonry.scss

@@ -1,35 +1,75 @@
-.masonry {
-  &.document {
-    padding: 0;
-    margin: 0 $spacer $spacer 0;
-    border-bottom: 0;
-    background-color: $gray-300;
-    min-height: 70px;
+.documents-masonry {
+  .document {
+    max-width: 25%;
+
+    &:hover, &:focus {
+      .caption-area {
+        display: block;
+      }
+    }
+
+    margin-bottom: $spacer;
+    position: relative;
+    width: auto;
+
+    .thumbnail-container {
+      position: relative;
+      min-height: 200px;
+      min-width: 200px;
+      background-color: $gray-300;
+    }
 
     .img-thumbnail {
-      border-radius: 0;
       padding: 0;
+      border-radius: 0;
     }
 
-    .caption {
-      &:first-child { display: block; } // To display captions when there is no image
-      a {
-        color: $gray-300;
+    .caption-area {
+      // styling for documents without thumbnails
+      &:first-child {
+        display: block;
+        max-height: 100%;
+        background-color: rgba(0,0,0,0.7) !important;
       }
-      line-height: 17px;
-      background-color: rgba(0,0,0,0.3);
-      text-align: center;
+      @extend .bg-dark;
+      @extend .text-white;
+      display: none;
+      padding: 5px 7px;
+      background-color: rgba(0,0,0,0.5) !important;
       position: absolute;
       bottom: 0;
-      right: 0;
-      left: 0;
+      overflow-x: hidden;
+      overflow-y: scroll;
+      max-height: 75%;
       width: 100%;
-      padding: 5px 7px;
+    }
+
+    .document-counter {
       display: none;
     }
-    &:hover {
-      .caption {
-        display: block;
+
+    .index_title {
+      @extend .h6;
+      a, a:hover, a:visited, a:active {
+        @extend .stretched-link;
+        color: $gray-300
+      }
+    }
+
+    .document-metadata {
+      dt {
+        color: $gray-300;
+      }
+      dt, dd {
+        flex: 0 0 100%;
+        max-width: 100%;
+        padding-left: 0;
+        padding-right: 0;
+        float: none;
+        width: auto;
+        clear: none;
+        text-align: left;
+        margin: 0;
       }
     }
   }

data/app/assets/stylesheets/blacklight_gallery/_slideshow.scss

@@ -101,7 +101,7 @@ $gray-dark: #343a40 !default;
   }
 }
 
-.slideshow-documents {
+.documents-slideshow {
   margin: 0;
 
   .info {
@@ -120,10 +120,12 @@ $gray-dark: #343a40 !default;
   }
 
   .grid {
+    display: flex;
+    flex-wrap: wrap;
     $square-thumb-size: 100px;
 
     .document {
-      float: left;
+      display: flex;
       margin-right: 20px;
       margin-top: 0;
       padding-top: 0;
@@ -140,7 +142,7 @@ $gray-dark: #343a40 !default;
         position: relative;
         width: $square-thumb-size;
 
-        a > img {
+        img {
           height: $square-thumb-size;
           max-height: none;
           max-width: none;

data/app/components/blacklight/gallery/document_component.html.erb

@@ -0,0 +1,21 @@
+<%= render(Blacklight::DocumentComponent.new(classes: 'col', component: :div, document: @document, counter: @counter)) do |component| %>
+  <% component.with(:body) do %>
+    <div class="thumbnail-container">
+      <% # Checking if #thumbnail takes any args before passing image_options. #thumbnail did not take any args in version 7.14.1 and earlier %>
+      <%= method(:thumbnail).arity.zero? ? thumbnail : thumbnail(class: 'img-thumbnail') %>
+
+      <div class="caption-area">
+        <div class="caption container">
+          <header class="documentHeader row">
+            <%= content_tag @title_component, class: 'index_title document-title-heading' do %>
+              <%= before_title %><%= title %><%= after_title %>
+            <% end %>
+            <%= actions %>
+          </header>
+          <%= content %>
+          <%= metadata %>
+        </div>
+      </div>
+    </div>
+  <% end %>
+<% end %>

data/app/components/blacklight/gallery/document_component.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module Blacklight
+  module Gallery
+    class DocumentComponent < Blacklight::DocumentComponent
+      def render_document_class(*args)
+        @view_context.render_document_class(*args)
+      end
+    end
+  end
+end

data/app/components/blacklight/gallery/slideshow_component.html.erb

@@ -0,0 +1,12 @@
+<div class="item<%= ' active' if @counter == 1 %>">
+  <div class="frame">
+      <%= slideshow_tag %>
+      <div class="caption">
+        <%= presenter.heading %>
+      </div>
+
+      <span class="counter">
+        <%= t :'blacklight_gallery.catalog.modal_slideshow.counter', counter: @counter, count: count %>
+      </span>
+  </div>
+</div>

data/app/components/blacklight/gallery/slideshow_component.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module Blacklight
+  module Gallery
+    class SlideshowComponent < Blacklight::DocumentComponent
+      def count
+        @document.response&.total
+      end
+
+      def render_document_class(*args)
+        @view_context.render_document_class(*args)
+      end
+
+      def presenter
+        @presenter ||= @view_context.document_presenter(@document)
+      end
+
+      def slideshow_tag(image_options = { alt: '' })
+        if view_config.slideshow_method
+          method_name = view_config.slideshow_method
+          @view_context.send(method_name, @document, image_options)
+        elsif view_config.slideshow_field
+          url = slideshow_image_url
+
+          image_tag url, image_options if url.present?
+        elsif presenter.thumbnail.exists?
+          presenter.thumbnail.thumbnail_tag(image_options, url_options.reverse_merge(suppress_link: true))
+        end
+      end
+
+      def slideshow_image_url
+        @document.first(view_config.slideshow_field) if @document.has? view_config.slideshow_field
+      end
+
+      def view_config
+        presenter.thumbnail.view_config
+      end
+    end
+  end
+end

data/app/components/blacklight/gallery/slideshow_preview_component.html.erb

@@ -0,0 +1,5 @@
+<%= render(Blacklight::DocumentComponent.new(classes: 'slideshow-preview-thumbnail', component: :div, document: @document, counter: @counter)) do |component| %>
+  <% component.with(:body) do %>
+    <%= @view_context.link_to_document(@document, thumbnail, class: 'thumbnail', data: { 'context-href': nil, 'slide-to': @document_counter - 1, toggle: "modal", target: "#slideshow-modal" }) %>
+  <% end %>
+<% end %>

data/app/components/blacklight/gallery/slideshow_preview_component.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module Blacklight
+  module Gallery
+    class SlideshowPreviewComponent < Blacklight::DocumentComponent
+      with_collection_parameter :document
+
+      def initialize(document:, document_counter: nil, **args)
+        super(document: document, document_counter: document_counter, **args)
+        @document_counter = document_counter || @counter
+      end
+
+      def thumbnail
+        @thumbnail ||
+          (presenter.thumbnail.exists? && presenter.thumbnail.render({ alt: presenter.heading })) ||
+          content_tag(:div, t('.missing_image', scope: [:blacklight_gallery]), class: 'thumbnail thumbnail-placeholder')
+      end
+
+      def presenter
+        @presenter ||= @view_context.document_presenter(@document)
+      end
+
+      def render_document_class(*args)
+        @view_context.render_document_class(*args)
+      end
+    end
+  end
+end

data/app/helpers/blacklight/gallery_helper.rb

@@ -1,42 +1,4 @@
 module Blacklight
   module GalleryHelper
-    def render_gallery_collection documents
-      index = -1
-      documents.map do |object|
-        index += 1
-        template = gallery_wrapper_template(object)
-        template.render(self, {document: object, document_counter: index}) if template
-
-      end.join().html_safe
-    end
-
-    def render_slideshow_tag(document, image_options = {}, url_options = {})
-      if blacklight_config.view_config(document_index_view_type).slideshow_method
-        method_name = blacklight_config.view_config(document_index_view_type).slideshow_method
-        send(method_name, document, image_options)
-      elsif blacklight_config.view_config(document_index_view_type).slideshow_field
-        url = slideshow_image_url(document)
-
-        image_tag url, image_options if url.present?
-      elsif has_thumbnail?(document)
-        render_thumbnail_tag(document, image_options, url_options.reverse_merge(suppress_link: true))
-      end
-    end
-
-    def slideshow_image_url(document)
-      if document.has? blacklight_config.view_config(document_index_view_type).slideshow_field
-        document.first(blacklight_config.view_config(document_index_view_type).slideshow_field)
-      end
-    end
-
-    def gallery_wrapper_template(object)
-      format = document_partial_name(object, nil)
-      ['index_gallery_%{format}_wrapper', 'index_gallery'].each do |str|
-        partial = str % { format: format }
-        logger.debug "Looking for gallery document wrapper #{partial}"
-        template = lookup_context.find_all(partial, lookup_context.prefixes, true, [:document, :document_counter], {}).first
-        return template if template
-      end
-    end
   end
 end