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

Don't let elements that follow questions float #62

Merged
merged 2 commits into from
May 1, 2015

Conversation

quis
Copy link
Contributor

@quis quis commented Apr 30, 2015

If a question contains floated elements then there is a risk that the following elements will float in an undesirable way. This change prevents that.

It also adds some spacing between questions to give them a bit more breathing room.

Example of the problem:

image

If a question contains floated elements then there is a risk that the
following elements will float in an undesirable way. This change
prevents that.

It also adds some spacing between questions to give them a bit more
breathing room.
@quis quis force-pushed the fix-floating-questions branch from 627c3e7 to ecde070 Compare May 1, 2015 08:33
@tombye
Copy link
Contributor

tombye commented May 1, 2015

Could we use contain-floats from the govuk front-end toolkit? It would mean we wouldn't need the overflow: hidden which might trip us up later at some point.

Use of overflow: hidden could trip us up later
@quis
Copy link
Contributor Author

quis commented May 1, 2015

@tombye f977457

tombye added a commit that referenced this pull request May 1, 2015
Don't let elements that follow questions float
@tombye tombye merged commit a737726 into master May 1, 2015
@tombye tombye deleted the fix-floating-questions branch May 1, 2015 09:04
quis added a commit that referenced this pull request May 1, 2015
Includes:
- #62 Don't let elements that follow questions float
@quis quis mentioned this pull request May 1, 2015
quis added a commit to Crown-Commercial-Service/digitalmarketplace-admin-frontend that referenced this pull request May 1, 2015
This commit:
- Adds Javascript for selection buttons
- Brings in the SASS changes from Crown-Commercial-Service/digitalmarketplace-frontend-toolkit#62
quis added a commit to Crown-Commercial-Service/digitalmarketplace-admin-frontend that referenced this pull request May 1, 2015
This commit:
- Adds Javascript for selection buttons
- Brings in the SASS changes from Crown-Commercial-Service/digitalmarketplace-frontend-toolkit#62
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