Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: paragraph width / math #374

Merged
merged 11 commits into from
Aug 28, 2019
Merged

fix: paragraph width / math #374

merged 11 commits into from
Aug 28, 2019

Conversation

alisonjoseph
Copy link
Member

@alisonjoseph alisonjoseph commented Aug 26, 2019

Fixes incorrect right spacing for blockquotes carbon-design-system/design-language-website#216

Fixes incorrect math for paragraph widths. (needs to be inside a row for % to be correct)

screenshot with some demo styles showing how things align correctly whether inside or outside column/rows

Screen Shot 2019-08-27 at 2 32 28 PM

@vercel
Copy link

vercel bot commented Aug 26, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://gatsby-theme-carbon-git-fork-alisonjoseph-paragraph-grid.carbon-design-system.now.sh

@vpicone
Copy link
Contributor

vpicone commented Aug 28, 2019

@jeanservaas Should the text be aligning with the second image here? @alisonjoseph mentioned that paragraphs typically have a 32px padding right but that seems to not necessarily be correct on the idl

this pr

Screen Shot 2019-08-28 at 11 17 49 AM

idl

Screen Shot 2019-08-28 at 11 22 07 AM

@vpicone vpicone requested a review from jeanservaas August 28, 2019 16:26
@vercel vercel bot temporarily deployed to staging August 28, 2019 16:52 Inactive
@vercel vercel bot temporarily deployed to staging August 28, 2019 16:56 Inactive
@alisonjoseph
Copy link
Member Author

confirmed w/ Jeanne over slack that we're good

…-carbon into paragraph-grid

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
…-carbon into paragraph-grid

* 'paragraph-grid' of github.com:alisonjoseph/gatsby-theme-carbon:
@vercel vercel bot temporarily deployed to staging August 28, 2019 17:32 Inactive
@vercel vercel bot temporarily deployed to staging August 28, 2019 17:34 Inactive
@vpicone vpicone merged commit 3131d40 into carbon-design-system:master Aug 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants