Merge pull request #241 from MITLibraries/use-65-tacos-interventions

Use 65 tacos interventions

Author: JPrevost

README.md

@@ -93,7 +93,6 @@ See `Optional Environment Variables` for more information.
 - `BOOLEAN_OPTIONS`: comma separated list of values to present to testers on instances where `BOOLEAN_PICKER` feature is enabled.
 - `BOOLEAN_PICKER`: feature to allow users to select their preferred boolean type. If set, feature is enabled. This feature is only intended for internal team
   testing and should never be enabled in production (mostly because the UI is a mess more than it would cause harm).
-- `FACT_PANELS_ENABLED`: Comma separated list of enabled fact panels. See `/views/results.html.erb` for implemented panels/valid options. Leave unset to disable all.
 - `FILTER_ACCESS_TO_FILES`: The name to use instead of "Access to files" for that filter / aggregation.
 - `FILTER_CONTENT_TYPE`: The name to use instead of "Content type" for that filter / aggregation.
 - `FILTER_CONTRIBUTOR`: The name to use instead of "Contributor" for that filter / aggregation.

app/assets/stylesheets/application.css.scss

@@ -19,3 +19,4 @@
 @import "partials/_shared";
 @import "partials/_results";
 @import "partials/_typography";
+@import "partials/_suggestion-panel";

app/assets/stylesheets/partials/_panels.scss

@@ -62,17 +62,6 @@
   }
 }
 
-.fact {
-  .panel {
-    margin-top: 0;
-
-    .panel-heading {
-      margin-top: 0;
-      padding-top: 16.8px;
-    }
-  }
-}
-
 .ask-us {
   margin-top: 3rem;
   a {

app/assets/stylesheets/partials/_suggestion-panel.scss

@@ -0,0 +1,106 @@
+/* Color Variables */
+
+// Core
+$blue-500: #0000FF;
+
+$purple-700: #990099;
+
+$white: #fff;
+$gray-100: #F2F2F2;
+$black: #111;
+
+@mixin focus-outline {
+    &:focus {
+        outline: 3px solid $blue-500;
+    }    
+}
+
+@mixin hover-transition {
+    transition: all .25s ease-in-out 0s;
+}
+
+// Semantic
+$color-suggestion-border: $purple-700;
+$color-suggestion-accent-text: $purple-700;
+
+/* Suggestion Panel */
+.mitlib-suggestion-panel {
+    border: 4px solid $color-suggestion-border;
+
+    display: flex;
+    column-gap: 24px;
+
+    padding: 20px 24px;
+    position: relative;
+
+    .panel-type {
+        color: $color-suggestion-accent-text;
+        font-size: 14px;
+        font-weight: 600;
+        margin-bottom: 8px;
+    }
+
+    h3 {
+        font-size: 20px;
+        font-weight: 600;
+        line-height: 1.25;
+        margin-bottom: 8px;
+    }
+
+    p {
+        font-size: 16px;
+    }
+
+    ul.metadata {
+        font-size: 14px;
+        list-style: none inside;
+        padding-left: 0;
+        margin-bottom: 20px;
+    }
+    
+    // This would need to be extracted into an Icon Button in the future.
+    button.dismiss {
+        //Temporarily hiding until we can move to a prop
+        display: none;
+
+        width: 48px;
+        height: 48px;
+        position: absolute;
+        top: 0;
+        right: 0;
+
+        background-color: $white;
+        border: none;
+        color: $black;
+        cursor: pointer;
+        font-weight: 600;
+
+	    @include hover-transition;
+
+        @include focus-outline;
+
+        &:hover {
+            background-color: $gray-100;
+        }
+    }
+}
+
+/* Button styles to be extracted into button component */
+.button.secondary {
+    border: 1px solid $black;
+    border-radius: 0;
+    display: inline-block;
+    padding: 6px 12px;
+    text-decoration: none;
+    font-size: 16px;
+    font-weight: 600;
+
+    @include hover-transition;
+
+    @include focus-outline;
+
+    &:hover {
+        color: $white;
+        background-color: $black;
+    }
+}

app/controllers/fact_controller.rb

@@ -4,6 +4,10 @@ class TacosController < ApplicationController
   def analyze
     return unless ApplicationHelper.tacos_enabled?
 
-    Tacos.analyze(params[:q])
+    tacos_response = Tacos.analyze(params[:q])
+
+    # Suggestions return as an array but we don't want to display more than one.
+    # We may want to have a "priority" system in the future to determine which suggestion to show.
+    @suggestions = tacos_response['data']['logSearchEvent']['detectors']['suggestedResources'].first
   end
 end

app/controllers/tacos_controller.rb

@@ -1,11 +1,4 @@
 module RecordHelper
-  def doi(metadata)
-    dois = metadata['identifiers']&.select { |id| id['kind'].downcase == 'doi' }
-    return unless dois.present?
-
-    dois.first['value']
-  end
-
   def date_parse(date)
     return unless date.present?
 

app/helpers/record_helper.rb

@@ -1,8 +1,4 @@
 module ResultsHelper
-  def fact_enabled?(fact_type)
-    ENV.fetch('FACT_PANELS_ENABLED', false).split(',').include?(fact_type)
-  end
-
   def results_summary(hits)
     hits.to_i >= 10_000 ? '10,000+ results' : "#{number_with_delimiter(hits)} results"
   end