Skip to content

Commit ba6ad41

Browse files
authored
Remove Block Alignment Override Styles for Deck (#1941)
This PR removes the custom block alignment styles for the Deck object. These weren't working quite as intended, and we're removing them in favor of editors adding the `o-container__pad` class as needed.
1 parent 4b78063 commit ba6ad41

File tree

3 files changed

+5
-37
lines changed

3 files changed

+5
-37
lines changed

.changeset/famous-mirrors-ring.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@cloudfour/patterns": patch
3+
---
4+
5+
Remove block alignment override styles for Deck

src/objects/deck/deck.scss

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -53,22 +53,3 @@
5353
.o-deck--align-start {
5454
align-items: start;
5555
}
56-
57-
/**
58-
* Block alignment adjustments
59-
*
60-
* 1. Set inline padding since we don't want the deck to touch the viewport
61-
* edges even when it's full bleed.
62-
* 2. Remove padding once `alignwide` is no longer full-bleed.
63-
*/
64-
65-
.o-deck.alignfull,
66-
.o-deck.alignwide {
67-
@include spacing.fluid-padding-inline; // 1
68-
}
69-
70-
.o-deck.alignwide {
71-
@media (width >= breakpoint.$l) {
72-
padding-inline: 0; // 2
73-
}
74-
}

src/objects/deck/deck.stories.mdx

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -122,24 +122,6 @@ The `o-deck--align-start` modifier can be used to top-align deck items. This mod
122122
</Story>
123123
</Canvas>
124124

125-
## Block Alignment
126-
127-
When used in WordPress, the `alignwide` and `alignfull` block alignment styles have been updated for use with the Deck.
128-
129-
<Canvas>
130-
<Story
131-
name="Block Alignment"
132-
height="400px"
133-
args={{
134-
columns: 3,
135-
columnsBreakpoint: '@m',
136-
alignment: 'alignfull',
137-
}}
138-
>
139-
{articlesStory.bind({})}
140-
</Story>
141-
</Canvas>
142-
143125
## Specifying Columns
144126

145127
While automatic columns are convenient, there are times when a specific column count is desired. For example, you may want to limit a design to three columns at larger breakpoints to align with adjacent elements.

0 commit comments

Comments
 (0)