Skip to content

Commit 979414b

Browse files
authored
fix: issue with drawer header not aligned correctly (#1453)
1 parent e6805f2 commit 979414b

2 files changed

Lines changed: 16 additions & 19 deletions

File tree

packages/components/src/components/drawer/drawer.lite.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,9 @@ export default function DBDrawer(props: DBDrawerProps) {
103103
data-direction={props.direction}
104104
data-rounded={props.rounded}>
105105
<header class="db-drawer-header">
106-
<Slot name="drawer-header" />
106+
<div class="db-drawer-header-text">
107+
<Slot name="drawer-header" />
108+
</div>
107109
<Show when={props.withCloseButton}>
108110
<DBButton
109111
className="button-close-drawer"

packages/components/src/components/drawer/drawer.scss

Lines changed: 13 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -53,11 +53,6 @@
5353
}
5454
}
5555

56-
%close-button-position {
57-
margin-inline-start: $db-spacing-fixed-lg;
58-
float: right;
59-
}
60-
6156
$spacings: (
6257
"small": (
6358
"block": $db-spacing-fixed-xs,
@@ -81,18 +76,12 @@ $spacings: (
8176
padding-block: map.get($spacing, "block");
8277

8378
.db-drawer-header {
84-
padding-block-end: var(
85-
--db-drawer-header-padding-block-end,
86-
map.get($spacing, "block")
87-
);
79+
padding-block-end: map.get($spacing, "block");
8880
padding-inline: map.get($spacing, "inline");
8981
}
9082

9183
.db-drawer-content {
92-
padding-inline: var(
93-
--db-drawer-content-padding-inline,
94-
map.get($spacing, "inline")
95-
);
84+
padding-inline: map.get($spacing, "inline");
9685
}
9786
}
9887

@@ -145,19 +134,25 @@ $spacings: (
145134

146135
.db-drawer-header {
147136
@include divider("bottom");
148-
display: block;
137+
display: none;
138+
139+
.db-drawer-header-text {
140+
margin-block: auto;
141+
font-weight: 700;
142+
}
149143

150144
.button-close-drawer {
151-
@extend %close-button-position;
145+
margin-inline-start: auto;
152146

153147
// angular-workaround
154148
button {
155-
@extend %close-button-position;
149+
margin-inline-start: auto;
156150
}
157151
}
158152

159-
&:not(:has(> :nth-child(1))) {
160-
display: none;
153+
&:has(.button-close-drawer),
154+
&:has(:not(.db-drawer-header-text:empty)) {
155+
display: flex;
161156
}
162157
}
163158
}

0 commit comments

Comments
 (0)