Skip to content

💄 Border heavy and spacing updates for Dialog#1198

Merged
mariush2 merged 3 commits into
mainfrom
feat/dialog-styling
Jan 7, 2026
Merged

💄 Border heavy and spacing updates for Dialog#1198
mariush2 merged 3 commits into
mainfrom
feat/dialog-styling

Conversation

@mariush2
Copy link
Copy Markdown
Contributor

@mariush2 mariush2 commented Jan 6, 2026

Azure DevOps links

User story


  • Needs to be tested locally by reviewer

Description

  • Border color heavy in dialog
  • Spacing adjustments in dialog

Copilot AI review requested due to automatic review settings January 6, 2026 08:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Dialog component styling to use heavier borders and adjust internal spacing for improved visual hierarchy and layout consistency.

  • Updates border colors from background__medium to background__heavy for more prominent visual separation
  • Adjusts DialogTitle padding to be asymmetric with different values for each side
  • Removes duplicate border-top declaration from DialogActions (keeping only the conditional version)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 6, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 100% (🎯 100%) 16614 / 16614
🔵 Statements 100% (🎯 100%) 16614 / 16614
🔵 Functions 100% (🎯 100%) 1200 / 1200
🔵 Branches 100% (🎯 100%) 4135 / 4135
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/molecules/Dialog/Dialog.styles.ts 100% 100% 100% 100%
Generated in workflow #2034 for commit 300946e by the Vitest Coverage Report Action

@mariush2 mariush2 marked this pull request as ready for review January 7, 2026 06:27
@mariush2 mariush2 requested a review from a team as a code owner January 7, 2026 06:27
@mariush2 mariush2 requested review from aslakihle and Copilot January 7, 2026 06:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mariush2 mariush2 merged commit ab40307 into main Jan 7, 2026
15 of 18 checks passed
@mariush2 mariush2 deleted the feat/dialog-styling branch January 7, 2026 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants