fix(accordion): remove max-width prop (#5332)

* fix(accordion): remove max-width prop * chore: remove on focus and add focus visible * chore: update fix for overflow * chore: update popover-brand variant button focus
salesforce-ux · Apr 29, 2024 · 7207364 · 7207364
1 parent 13ff5f6
commit 7207364
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/ui/components/accordion/base/_index.scss b/ui/components/accordion/base/_index.scss
@@ -57,6 +57,11 @@
   padding-bottom: var(--slds-c-accordion-section-spacing-block-end, var(--sds-c-accordion-section-spacing-block-end, #{$spacing-small}));
   padding-left: var(--slds-c-accordion-section-spacing-inline-start, var(--sds-c-accordion-section-spacing-inline-start, #{$spacing-small}));
   background-color: var(--slds-c-accordion-summary-color-background, var(--sds-c-accordion-summary-color-background, transparent));
+
+  // For Action Overflow Menu variant
+  .slds-dropdown-trigger .slds-button {
+    margin-left: $spacing-small;
+  }
 }
 
 /**
@@ -112,7 +117,6 @@
   &:hover,
   &:focus {
     color: var(--sds-c-accordion-heading-text-color-hover, var(--sds-c-accordion-heading-text-color-hover, $brand-accessible-active));
-    max-width: max-content;
   }
 }
 

diff --git a/ui/components/popovers/brand/_index.scss b/ui/components/popovers/brand/_index.scss
@@ -31,8 +31,8 @@
     }
   }
 
-  // Historically Close button is not visible hence removed the focus
-  &:not(.slds-popover_brand-dark) {
+  // Historically close button is not visible for Brand variant
+  &:not(.slds-popover_brand-dark):not(.slds-popover_prompt) {
     .slds-button:focus {
       box-shadow: none;
     }