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

Css padding fixes #935

Merged
merged 2 commits into from
May 2, 2019
Merged

Conversation

jcushman
Copy link
Contributor

@jcushman jcushman commented May 2, 2019

This should get margins looking right on the /terms page, and adjust the margins between sections on all the other pages.

Terms:

image

Other pages using h2.subtitle for section headers:

image

Gallery:

image

Specific changes in the PR:

  • Refactor some scss files to avoid redundant imports. Mostly that was by moving a mixin for the colored-footer files from base.scss to variables.scss. One exception, index.scss has to import base.scss, so I tweaked main_base.html and index.html to make sure that base.scss itself wasn't also included on the homepage.

  • Remove the <div class="page-section"> element (almost all the changes here are just removing that element and reindenting).

  • Move the margin-top and margin-bottom that were on .page-section to be on .main-content and h2.subtitle instead.

  • Custom padding in gallery.scss since it wasn't using h2.subtitle for spacing between projects

@bensteinberg bensteinberg merged commit 6fb66c0 into harvard-lil:develop May 2, 2019
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.

None yet

2 participants