Skip to content

Conversation

@spaceninja
Copy link
Member

Overview

This PR adds the new o-container__pad class, which behaves similarly
to the existing o-container__fill & o-container__fill-pad classes,
adding the standard container inline padding to a container child.

Screenshots

Screen Shot 2022-07-12 at 3 10 38 PM

Testing

  1. Review the Container > Fill story on the preview deploy

This PR adds the new `o-container__pad` class, which behaves similarly
to the existing `o-container__fill` & `o-container__fill-pad` classes,
adding the standard container inline padding to a container child.

Fixes #1935
@spaceninja spaceninja requested a review from tylersticka July 12, 2022 22:11
@changeset-bot
Copy link

changeset-bot bot commented Jul 12, 2022

🦋 Changeset detected

Latest commit: 9e04929

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cloudfour/patterns Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link

netlify bot commented Jul 12, 2022

Deploy Preview for cloudfour-patterns ready!

Name Link
🔨 Latest commit 9e04929
🔍 Latest deploy log https://app.netlify.com/sites/cloudfour-patterns/deploys/62cdf4ced3e5ce0008011c49
😎 Deploy Preview https://deploy-preview-1938--cloudfour-patterns.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@spaceninja spaceninja marked this pull request as ready for review July 12, 2022 22:11
Co-authored-by: Tyler Sticka <tyler@cloudfour.com>
@spaceninja spaceninja requested a review from tylersticka July 12, 2022 22:25
@spaceninja spaceninja merged commit adae056 into v-next Jul 12, 2022
@spaceninja spaceninja deleted the chore/add-o-container__pad branch July 12, 2022 22:53
@github-actions github-actions bot mentioned this pull request Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add o-container__pad element class

3 participants