Skip to content

Commit c32d6bc

Browse files
authored
refactor: add missing no-padding variant to dialog base styles (#9843)
1 parent 58b056d commit c32d6bc

File tree

3 files changed

+13
-0
lines changed

3 files changed

+13
-0
lines changed

packages/dialog/src/styles/vaadin-dialog-overlay-base-styles.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,10 @@ export const dialogOverlayBase = css`
6464
padding: var(--vaadin-dialog-padding, var(--vaadin-padding));
6565
}
6666
67+
:host([theme~='no-padding']) [part='content'] {
68+
padding: 0 !important;
69+
}
70+
6771
:host(:is([has-header], [has-title])) [part='content'] {
6872
padding-top: 0;
6973
}

packages/dialog/test/visual/base/dialog.test.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,4 +74,13 @@ describe('dialog', () => {
7474
await nextUpdate(element);
7575
await visualDiff(div, 'header-title-long-single-word');
7676
});
77+
78+
it('no-padding theme', async () => {
79+
element.setAttribute('theme', 'no-padding');
80+
const contentStyles = new CSSStyleSheet();
81+
contentStyles.insertRule('vaadin-dialog::part(content) { padding: 20px; }');
82+
document.adoptedStyleSheets = [contentStyles];
83+
await nextUpdate(element);
84+
await visualDiff(div, 'content-no-padding-theme');
85+
});
7786
});
6.67 KB
Loading

0 commit comments

Comments
 (0)