fix: pfe-accordion disclosure (#1003)

* fix: Disclosure hover fix
* fix: Update chevron
* fix: Fix sizing flicker on border when disclosure is opened/closed

Author: castastrophe

CHANGELOG-prerelease.md

@@ -1,3 +1,7 @@
+## Prerelease 54 ( TBD )
+
+- []() fix: pfe-accordion border styles (#1002)
+
 ## Prerelease 53 ( 2020-07-21 )
 
 - [743913e](https://github.com/patternfly/patternfly-elements/commit/743913e0620ee0ca6f0cdf95cc1ed2597b5fe6e3) fix: pfe-autocomplete prevent search-event from firing twice on option select (#989)

elements/pfe-accordion/src/pfe-accordion-header.scss

@@ -81,18 +81,20 @@
       border-left-width: pfe-fetch(surface--border-width);
     }
 
-    &:hover {
+    &::before {
+      @include pfe-chevron($state: closed, $position: before, $offset: .55em);
+    }
+    
+    &[aria-expanded="true"]::before {
+      transform: rotate(45deg);
+    }
+    
+    &[aria-expanded="true"],
+    &:not([aria-expanded="true"]):hover {
       padding-left: calc(#{pfe-local(Padding, $region: base)} * 3 - #{pfe-var(surface--border-width--heavy)} + #{pfe-var(surface--border-width)});
       &::before {
         margin-left: calc((#{pfe-var(surface--border-width--heavy)} - #{pfe-var(surface--border-width)}) * -1);
       }
     }
-
-    &::before {
-      @include pfe-chevron($state: open, $position: before);
-    }
-    &[aria-expanded="true"]::before {
-      transform: rotate(45deg) translateY(4px);
-    }
   }
 }

elements/pfe-sass/mixins/_components.scss

@@ -87,7 +87,7 @@
 @mixin pfe-accordion-props($variant: default) {
   margin: 0;
   width: pfe-local(Width, 100%);
-  max-width: calc(100% - #{pfe-local(BorderLeftWidth)});
+  max-width: calc(100% - #{pfe-var(surface--border-width--heavy)});
   height: auto;
   position: relative;