more css changes and fixed an issue on eds target

css/roblib_search.base.css

@@ -12,32 +12,47 @@
 .panel-pane {border-top:1px solid #cecfcc;
             border-left:1px solid #cecfcc; border-right:1px solid #cecfcc; line-height: 1.5em; margin:10px; border-bottom:1px solid #cecfcc;
 }
-
+.roblib-search-header {
+    background:#333;
+    padding-left: 5px;
+    border-left:dotted;
+    color:white;
+    font-weight:lighter;
+    font-family:Verdana,Arial, helvetica, sans-serif;
+}
 .roblib-search-eds-media-header {
-    background:indianred;
+    border-color:indianred;
+    border-left-width: 10px;
 }
 .roblib-search-eds-articles-header {
-    background:lightcoral;
+    border-color:indianred;
+    border-left-width: 10px;
 }
 .roblib-search-cufts-header {
-    background:lightblue;
+    border-color:lightblue;
+    border-left-width: 10px;
 }
 .roblib-search-solr-results-header {
-    background:lightsalmon;
+    border-color:lightsalmon;
+    border-left-width: 10px;
 }
 .roblib-search-solr-db-header {
-    background:darkorange;
+    border-left-color:darkorange;
+    border-left-width: 10px;
 }
 .roblib-search-solr-bestbet-header {
-    background:orange;
+    border-left-color:orange;
+    border-left-width: 10px;
 }
 
 
 .roblib-search-islandscholar-header {
-     background:darkolivegreen;
+    border-left-color:darkolivegreen;
+    border-left-width: 10px;
  }
 .roblib-search-evergreen-header {
-    background:forestgreen;
+    border-left-color:forestgreen;
+    border-left-width: 10px;
 }
 
 .roblib-search-row {

targets/cufts/roblib_search_cufts.module

@@ -122,7 +122,7 @@ function roblib_search_cufts_theme() {
 
 function roblib_search_cufts_preprocess_panels_pane(&$variables, $hook){
   if($variables['pane']->subtype == 'roblib_search_cufts-roblib_search_cufts_results'){
-    $variables['title_prefix'] = '<div class="roblib-search-cufts-header">';
+    $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-cufts-header">';
     $variables['title_suffix'] ='</div>';
   }
 }

targets/eds/js/eds_results.js

@@ -72,11 +72,15 @@ Drupal.behaviors.roblib_search_eds = {
 
 function edsPopulatePopupDivs(content, val, counter){
     content[counter] = '';
+    try{
         jQuery.each(val.DetailedRecord, function(key, value){
             if(value.Group == 'Note' || value.Group == 'TOC') {
                 content[counter] += '<div class="eds-popup-content"><span class="eds-popup-label">' + value.Label  + '</span>' ;
                 content[counter] += '<span class="eds-popup-value"> ' + value.Data +'</span></div>';
             }
         })
+    } catch (err) {
+        // do nothing as there is no data to work with
+    }
 }
 

targets/eds/roblib_search_eds.module

@@ -153,11 +153,11 @@ function roblib_search_eds_theme() {
 
 function roblib_search_eds_preprocess_panels_pane(&$variables, $hook){
   if($variables['pane']->subtype == 'roblib_search_eds-roblib_search_eds_articles'){
-    $variables['title_prefix'] = '<div class="roblib-search-eds-articles-header">';
+    $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-eds-articles-header">';
     $variables['title_suffix'] ='</div>';
   }
   if($variables['pane']->subtype == 'roblib_search_eds-roblib_search_eds_results'){
-    $variables['title_prefix'] = '<div class="roblib-search-eds-media-header">';
+    $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-eds-media-header">';
     $variables['title_suffix'] ='</div>';
   }
 }

targets/evergreen/roblib_search_evergreen.module

@@ -97,7 +97,7 @@ function roblib_search_evergreen_permission() {
 
 function roblib_search_evergreen_preprocess_panels_pane(&$variables, $hook){
   if($variables['pane']->subtype == 'roblib_search_evergreen-roblib_search_evergreen_results'){
-    $variables['title_prefix'] = '<div class="roblib-search-evergreen-header">';
+    $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-evergreen-header">';
     $variables['title_suffix'] ='</div>';
   }
 }

targets/islandscholar/roblib_search_islandscholar.module

@@ -100,7 +100,7 @@ function roblib_search_islandscholar_block_info() {
 
 function roblib_search_islandscholar_preprocess_panels_pane(&$variables, $hook){
   if($variables['pane']->subtype == 'roblib_search_islandscholar-roblib_islandscholar_results'){
-    $variables['title_prefix'] = '<div class="roblib-search-islandscholar-header">';
+    $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-islandscholar-header">';
     $variables['title_suffix'] ='</div>';
   }
 }

targets/solr_site/roblib_search_solr_site.module

@@ -165,22 +165,22 @@ function roblib_search_solr_site_block_info() {
 function roblib_search_solr_site_preprocess_panels_pane(&$variables, $hook){
   switch ($variables['pane']->subtype) {
     case 'roblib_search_solr_site-roblib_search_solr_site_bestbet':
-      $variables['title_prefix'] = '<div class="roblib-search-solr-bestbet-header">';
+      $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-solr-bestbet-header">';
       $variables['title_suffix'] = '</div>';
       break;
 
     case 'roblib_search_solr_site-roblib_search_solr_site_db':
-      $variables['title_prefix'] = '<div class="roblib-search-solr-db-header">';
+      $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-solr-db-header">';
       $variables['title_suffix'] = '</div>';
       break;
 
     case 'roblib_search_solr_site-roblib_search_solr_site_guides':
-      $variables['title_prefix'] = '<div class="roblib-search-solr-guides-header">';
+      $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-solr-guides-header">';
       $variables['title_suffix'] = '</div>';
       break;
 
     case 'roblib_search_solr_site-roblib_search_solr_site_results':
-      $variables['title_prefix'] = '<div class="roblib-search-solr-results-header">';
+      $variables['title_prefix'] = '<div class="roblib-search-header roblib-search-solr-results-header">';
       $variables['title_suffix'] = '</div>';
       break;
   }