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 alignment of report-a-problem at certain widths #660

Merged
merged 1 commit into from
Oct 9, 2015

Conversation

dsingleton
Copy link
Contributor

At a breakpoint below 1020 the report a problem link, and form, weren't aligned to the grid of the rest of the page, and were indented by an additional 30px on each side.

This was caused by %site-width-container being used on the report a problem containers, within #wrapper, which already extends %site-width-container, causing double margins.

As the report a problem elements already inside a page width container it's same to remove the use of %site-width-container for them.

🌟 Before After
Closed report-before report-after
Open report2-before report2-after
Mobile report3-before report3-after

At a breakpoint below 1020 the report a problem link, and
form, weren't aligned to the grid of the rest of the page, and were
indented by an additional 30px on each side.

This was caused by `%site-width-container` being used on the report
a problem containers, within `#wrapper`, which already extends
`%site-width-container`, causing double margins.

As the report a problem elements already inside a page width container
it's same to remove the use of `%site-width-container` for them.
@fofr
Copy link
Contributor

fofr commented Oct 9, 2015

👍

fofr added a commit that referenced this pull request Oct 9, 2015
Fix alignment of report-a-problem at certain widths
@fofr fofr merged commit 6fa1eb5 into master Oct 9, 2015
@fofr fofr deleted the fix-report-a-problem-alignment branch October 9, 2015 09:04
fofr added a commit that referenced this pull request Oct 12, 2015
…ignment"

This reverts commit 6fa1eb5, reversing
changes made to 52a5f91.
dsingleton added a commit that referenced this pull request Oct 12, 2015
…nt-fix

Revert "Merge pull request #660 from alphagov/fix-report-a-problem-alignment"
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.

2 participants