refs #23830 [adlr_link] add better focus on tabbing trough sidebar facets

* replace $database-link-hover-color (color not part of color schema) by $link-hover-color (=azure)
* remove style from exclude-icon -> applied styles from parent now working
* add focus to facet group toggler
* out of topic: rewrite fieldset-border

themes/fid_adlr/scss/_customTheme.scss

@@ -56,7 +56,7 @@ ul {
         color: $asphalt;
 
         &:hover {
-          color: $database-link-hover-color;
+          color: $link-hover-color;
         }
       }
     }

themes/fid_adlr/scss/_customVariables.scss

@@ -71,7 +71,6 @@ $state-success-bg: $brand-info !default;
 $state-info-bg: $brand-info !default;
 $state-info-text: $asphalt !default;
 $state-info-border: $asphalt !default;
-$database-link-hover-color: #00359b;
 $header-bg-color: $midnight;
 $a-visited-color: $black !default;
 $form-dropdown-focus-hover-color: #262626 !default;
@@ -187,7 +186,7 @@ $border-default-styles: 0px;
 // **** C0 **** Default anchor/link styles general *****************
 // *****************************************************************
 //$link-color: $brand-primary !default;
-//$link-hover-color: darken($link-color, 15%) !default;
+$link-hover-color: $azure;
 $link-text-decoration: none;
 //$link-hover-decoration: underline !default;
 //$link-on-dark-bg-color: invert($link-color) !default;
@@ -226,6 +225,8 @@ $facet-range-form-padding: 0;
 
 $form-control-height: em(60px);
 
+$fielset-default-border: 1px solid $asphalt-dark;
+
 
 // *****************************************************************
 // **** K5 **** Header and navbar  *********************************
@@ -385,6 +386,7 @@ $sidebar-item-active-hover-bg: none;
 
 $sidebar-or-facet-text-indent: 0;
 $sidebar-facet-active-text-inside-padding-left: 0;
+$sidebar-facet-active-hover-color: $link-hover-color;
 
 //// Sidebar item padding
 //$sidebar-item-padding: .75em 1em !default;

themes/fid_adlr/scss/components/_sidebar.scss

@@ -53,8 +53,16 @@
         border-bottom: 2px solid $violet;
       }
 
+      .all-filters,
       .remove-filter {
         color: $violet;
+
+        &:hover,
+        &:active,
+        &:focus,
+        {
+          color: $azure;
+        }
       }
 
       .date-fields {
@@ -96,6 +104,16 @@
     }
   }
 
+  // #23830 - add focus to facet group toggler
+  button.title {
+    &:hover,
+    &:active,
+    &:focus,
+    {
+      background-color: $seafoam;
+    }
+  }
+
   .active-filters,
   .switchtab-info {
     @include default-border($violet !important);
@@ -246,25 +264,26 @@
   margin-left: 0;
 }
 
+// #23830 - remove all styles from nested i ...
 .exclude i {
-  color: $black;
+  color: revert;
 
   &:hover,
   &:link,
   &:active,
   &:focus,
   &:visited {
-    color: $black;
+    color: revert;
   }
 }
 
-a {
+// #23830 - ... to apply styles properly to exclude link
+a.exclude {
   // #17913 adjust BARF to adlr design
-  &.exclude {
-    &:hover,
-    &:focus {
-      outline: none;
-    }
+  &:hover,
+  &:active,
+  &:focus {
+    outline: none;
   }
 }
 

themes/fid_adlr/templates/Recommend/SideFacets/single-facet.phtml

@@ -87,7 +87,8 @@
     <?php $excludeURL = $this->urlBase . $this->url->addFacet($this->group, $this->facet['value'], 'NOT'); ?>
     <?php /* finc: Barf for exclude facets #19392 */ ?>
     <a href="<?=$excludeURL ?>" data-lightbox-ignore class="exclude" title="<?= $this->transEsc('exclude_filter', ['%%filter_name%%' => $this->facet['displayText']]) ?>, <?= $this->transEsc('page_reload_on_exclude_hint', ['%%filter_name%%' => $this->facet['displayText']]) ?>">
-      <i class="fa fa-times" aria-hidden="true"></i>
+      <?php // add adlr.link facet remove icon - #23830 ?>
+      <i class="icon icon-minus-circle" aria-hidden="true"></i>
       <span class="sr-only"><?=$this->transEsc('exclude_filter', ['%%filter_name%%' => $this->facet['displayText']]) ?>, <?= $this->transEsc('page_reload_on_exclude_hint', ['%%filter_name%%' => $this->facet['displayText']]) ?></span>
     </a>
   <?php endif; ?>