Skip to content

Commit d6b2198

Browse files
committed
Fix divider and content order when flowing as row with a custom sidebar placement
1 parent 01bdfbf commit d6b2198

File tree

2 files changed

+29
-4
lines changed

2 files changed

+29
-4
lines changed

src/layout/layout.scss

+10
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,16 @@
128128
.Layout-main {
129129
grid-column: 3 / span 1;
130130
}
131+
132+
&.Layout--sidebarPosition-end {
133+
.Layout-sidebar {
134+
grid-column: 3 / span 1;
135+
}
136+
137+
.Layout-main {
138+
grid-column: 1;
139+
}
140+
}
131141
}
132142
}
133143

src/layout/mixins.scss

+19-4
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,6 @@
1111
grid-column: 1 !important;
1212
}
1313

14-
&.Layout--divided {
15-
@include flow-as-row-divider;
16-
}
17-
1814
&.Layout--sidebarPosition-flowRow-start {
1915
.Layout-sidebar {
2016
grid-row: 1;
@@ -40,13 +36,32 @@
4036
display: none;
4137
}
4238
}
39+
40+
&.Layout--divided {
41+
@include flow-as-row-divider;
42+
43+
.Layout-main {
44+
grid-row: 3 / span 1;
45+
}
46+
47+
&.Layout--sidebarPosition-flowRow-end {
48+
.Layout-sidebar {
49+
grid-row: 3 / span 1;
50+
}
51+
52+
.Layout-main {
53+
grid-row: 1;
54+
}
55+
}
56+
}
4357
}
4458

4559
@mixin flow-as-row-divider {
4660
--Layout-gutter: 0;
4761

4862
.Layout-divider {
4963
height: 1px;
64+
grid-row: 2;
5065

5166
&.Layout-divider--flowRow-hidden {
5267
display: none;

0 commit comments

Comments
 (0)