From c6b1b9ae61db8c1d25c65dee03fc8d737c75c01b Mon Sep 17 00:00:00 2001 From: vadim Date: Wed, 26 Mar 2025 16:48:08 +0100 Subject: [PATCH] chore: Fix button position and left padding of WDS modal (#39941) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Description Fixes #39767 | Before | After | |--------|--------| |Screenshot 2025-03-26 at 15 41 37|Screenshot 2025-03-26 at 15 58 30| |Screenshot 2025-03-26 at 15 41 40|Screenshot 2025-03-26 at 15 58 34| |Screenshot 2025-03-26 at 15 43 25|Screenshot 2025-03-26 at 15 55 57| ## Automation /ok-to-test tags="@tag.Sanity" ### :mag: Cypress test results > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: > Commit: 3d64e014ff432460b456471ed9eb3a0041e40754 > Cypress dashboard. > Tags: `@tag.Sanity` > Spec: >
Wed, 26 Mar 2025 15:44:12 UTC ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No ## Summary by CodeRabbit - **Style** - Refined the modal's visual layout for enhanced spacing and alignment. - Adjusted horizontal padding and inter-element gaps in the modal content. - Updated header margins and improved button vertical positioning for a more polished interface. --- .../src/components/Modal/src/styles.module.css | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/app/client/packages/design-system/widgets/src/components/Modal/src/styles.module.css b/app/client/packages/design-system/widgets/src/components/Modal/src/styles.module.css index 9e80a71f9e..deb772515d 100644 --- a/app/client/packages/design-system/widgets/src/components/Modal/src/styles.module.css +++ b/app/client/packages/design-system/widgets/src/components/Modal/src/styles.module.css @@ -14,16 +14,19 @@ outline: none; display: flex; flex-direction: column; - padding-inline: var(--outer-spacing-4); + padding-inline: var(--outer-spacing-5) var(--outer-spacing-4); padding-block: var(--outer-spacing-4); - gap: var(--outer-spacing-2); + gap: var(--outer-spacing-3); flex: 1; } .overlay .content .header { /* Needed to align the close button */ - padding-inline: 0 calc(var(--outer-spacing-2)); - margin-inline: 0 calc(-1 * var(--outer-spacing-4)); + margin-inline: 0 calc(-0.5 * var(--inner-spacing-3)); +} + +.overlay .content .header button { + margin-block: calc(-0.5 * var(--inner-spacing-3)) 0; } .overlay .content .body {