Text band margins #963

Merged
merged 1 commit into from Feb 22, 2016

Conversation

Projects
None yet
2 participants
@skoolbus39
Member

skoolbus39 commented Feb 22, 2016

The text-band should be centered with a max-width of 50rem, but using margin: 0 auto; to center the content removes the left and right margins, resulting in text that runs right up to the viewport edge. This commit keeps the margins at narrower browser widths (where the length of the text never exceeds 50rem), and instead removes them and applies the 50rem max-width at 960px, where the viewport exceeds the width of the content and margins combined.

Text band margins
The text-band should be centered with a max-width of 50rem, but using margin: 0 auto; to center the content removes the left and right margins, resulting in text that runs right up to the viewport edge. This commit keeps the margins at narrower browser widths (where the length of the text never exceeds 50rem), and instead removes them and applies the 50rem max-width at 960px, where the viewport exceeds the width of the content and margins combined.

kabel added a commit that referenced this pull request Feb 22, 2016

@kabel kabel merged commit 2697486 into unl:develop Feb 22, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment