Skip to content

Commit 364418a

Browse files
Jeffrey Lauwersclaude
andcommitted
fix(ModalDialog): spacing tokens aanpassen
- header padding-block-start: 3xl (24px) → xl (16px) - header padding-inline: 3xl (24px) → xl (16px) - body padding-block: xl (16px) → 3xl (24px) - body padding-inline: 3xl (24px) → xl (16px) - footer padding-inline: 3xl (24px) → xl (16px) - footer padding-block-end: nieuw eigen token (xl, 16px) voorheen werd header-padding-block-start hergebruikt CSS footer gebruikt nu --dsn-modal-dialog-footer-padding-block-end. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 8cb055d commit 364418a

3 files changed

Lines changed: 24 additions & 18 deletions

File tree

packages/components-html/src/modal-dialog/modal-dialog.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@
189189
flex-shrink: 0;
190190

191191
padding-block-start: var(--dsn-modal-dialog-footer-padding-block);
192-
padding-block-end: var(--dsn-modal-dialog-header-padding-block-start);
192+
padding-block-end: var(--dsn-modal-dialog-footer-padding-block-end);
193193
padding-inline: var(--dsn-modal-dialog-footer-padding-inline);
194194

195195
/* Scheidingslijn boven de footer */

packages/design-tokens/src/tokens/components/modal-dialog.json

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -60,43 +60,48 @@
6060
},
6161
"header": {
6262
"padding-block-start": {
63-
"value": "{dsn.space.block.3xl}",
63+
"value": "{dsn.space.block.xl}",
6464
"type": "spacing",
65-
"comment": "Bovenkant padding van de header (24px)"
65+
"comment": "Bovenkant padding van de header (16px)"
6666
},
6767
"padding-block-end": {
6868
"value": "{dsn.space.block.xl}",
6969
"type": "spacing",
7070
"comment": "Onderkant padding van de header (16px)"
7171
},
7272
"padding-inline": {
73-
"value": "{dsn.space.inline.3xl}",
73+
"value": "{dsn.space.inline.xl}",
7474
"type": "spacing",
75-
"comment": "Horizontale padding van de header (24px)"
75+
"comment": "Horizontale padding van de header (16px)"
7676
}
7777
},
7878
"body": {
7979
"padding-block": {
80-
"value": "{dsn.space.block.xl}",
80+
"value": "{dsn.space.block.3xl}",
8181
"type": "spacing",
82-
"comment": "Verticale padding van de body (16px)"
82+
"comment": "Verticale padding van de body (24px)"
8383
},
8484
"padding-inline": {
85-
"value": "{dsn.space.inline.3xl}",
85+
"value": "{dsn.space.inline.xl}",
8686
"type": "spacing",
87-
"comment": "Horizontale padding van de body (24px)"
87+
"comment": "Horizontale padding van de body (16px)"
8888
}
8989
},
9090
"footer": {
9191
"padding-block": {
9292
"value": "{dsn.space.block.xl}",
9393
"type": "spacing",
94-
"comment": "Verticale padding van de footer (16px)"
94+
"comment": "Bovenkant padding van de footer (16px)"
95+
},
96+
"padding-block-end": {
97+
"value": "{dsn.space.block.xl}",
98+
"type": "spacing",
99+
"comment": "Onderkant padding van de footer (16px)"
95100
},
96101
"padding-inline": {
97-
"value": "{dsn.space.inline.3xl}",
102+
"value": "{dsn.space.inline.xl}",
98103
"type": "spacing",
99-
"comment": "Horizontale padding van de footer (24px)"
104+
"comment": "Horizontale padding van de footer (16px)"
100105
}
101106
}
102107
}

packages/storybook/src/ModalDialog.docs.md

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -74,13 +74,14 @@ Het ModalDialog component toont een tijdelijk overlay-venster dat de achtergrond
7474
| `--dsn-modal-dialog-heading-font-size` | Tekstgrootte van de heading |
7575
| `--dsn-modal-dialog-heading-line-height` | Regelafstand van de heading |
7676
| `--dsn-modal-dialog-heading-color` | Kleur van de heading |
77-
| `--dsn-modal-dialog-header-padding-block-start` | Bovenkant padding van de header (24px) |
77+
| `--dsn-modal-dialog-header-padding-block-start` | Bovenkant padding van de header (16px) |
7878
| `--dsn-modal-dialog-header-padding-block-end` | Onderkant padding van de header (16px) |
79-
| `--dsn-modal-dialog-header-padding-inline` | Horizontale padding van de header (24px) |
80-
| `--dsn-modal-dialog-body-padding-block` | Verticale padding van de body (16px) |
81-
| `--dsn-modal-dialog-body-padding-inline` | Horizontale padding van de body (24px) |
82-
| `--dsn-modal-dialog-footer-padding-block` | Verticale padding van de footer (16px) |
83-
| `--dsn-modal-dialog-footer-padding-inline` | Horizontale padding van de footer (24px) |
79+
| `--dsn-modal-dialog-header-padding-inline` | Horizontale padding van de header (16px) |
80+
| `--dsn-modal-dialog-body-padding-block` | Verticale padding van de body (24px) |
81+
| `--dsn-modal-dialog-body-padding-inline` | Horizontale padding van de body (16px) |
82+
| `--dsn-modal-dialog-footer-padding-block` | Bovenkant padding van de footer (16px) |
83+
| `--dsn-modal-dialog-footer-padding-block-end` | Onderkant padding van de footer (16px) |
84+
| `--dsn-modal-dialog-footer-padding-inline` | Horizontale padding van de footer (16px) |
8485

8586
## Accessibility
8687

0 commit comments

Comments
 (0)