diff --git a/src/styles/components/chat/_chat-item-card.scss b/src/styles/components/chat/_chat-item-card.scss index 47e3cb73..d6a4c5ad 100644 --- a/src/styles/components/chat/_chat-item-card.scss +++ b/src/styles/components/chat/_chat-item-card.scss @@ -457,8 +457,27 @@ display: none; flex-flow: column nowrap; } - &.show-summary > .mynah-chat-item-card-summary-collapsed-content { - display: flex; + // When expanded, the header and the collapsed content should read as a + // single container instead of separate rounded boxes. Square the header's + // bottom corners and the content's top corners so they connect, and let the + // header's bottom border act as the divider between the header and the + // parameters/results below. + &.show-summary { + > .mynah-chat-item-card-summary-content { + border-bottom-left-radius: 0; + border-bottom-right-radius: 0; + } + > .mynah-chat-item-card-summary-collapsed-content { + display: flex; + border-right: var(--mynah-border-width) solid var(--mynah-color-border-default); + border-bottom: var(--mynah-border-width) solid var(--mynah-color-border-default); + border-left: var(--mynah-border-width) solid var(--mynah-color-border-default); + border-bottom-left-radius: var(--mynah-card-radius); + border-bottom-right-radius: var(--mynah-card-radius); + // Clip the inner parameter/result cards to the container's shape so + // their own corners do not create separate rounded boxes. + overflow: hidden; + } } } } diff --git a/ui-tests/__test__/flows/quick-action-commands-header.ts b/ui-tests/__test__/flows/quick-action-commands-header.ts index c90e28b6..a0e6c7ae 100644 --- a/ui-tests/__test__/flows/quick-action-commands-header.ts +++ b/ui-tests/__test__/flows/quick-action-commands-header.ts @@ -137,9 +137,9 @@ export const verifyQuickActionCommandsHeaderStatusVariations = async (page: Page let foundStatusClass = false; for (const statusClass of statusClasses) { - const hasStatus = await headerElement.evaluate((el, className) => + const hasStatus = Boolean(await headerElement.evaluate((el, className) => el.classList.contains(className), statusClass - ); + )); if (hasStatus) { foundStatusClass = true; console.log(`Found status class: ${statusClass}`);