From 7207364c5b6719ac2ac0f2317446755977aaa630 Mon Sep 17 00:00:00 2001 From: Amarnath Chandana Date: Mon, 29 Apr 2024 19:43:21 +0530 Subject: [PATCH] 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 --- ui/components/accordion/base/_index.scss | 6 +++++- ui/components/popovers/brand/_index.scss | 4 ++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/ui/components/accordion/base/_index.scss b/ui/components/accordion/base/_index.scss index 0f4fd8fff..9da9981bc 100644 --- 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 index e58743b98..539deb194 100644 --- 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; }