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 table spacing #482

Merged
merged 2 commits into from
Jun 5, 2017
Merged

Fix table spacing #482

merged 2 commits into from
Jun 5, 2017

Conversation

selfthinker
Copy link
Contributor

@selfthinker selfthinker commented Jun 2, 2017

Remove right padding from last table cells.
I also removed the color declaration as it isn't necessary.

What problem does the pull request solve?

This is a design-y problem. Can a designer please fill this section in and explain what the problem is? @edwardhorsford raised it.

Ed: it makes things look neater, and matches the padding (or lack thereof) used on the other side.

Screenshots

Before

spacing-before

After

spacing-after

What type of change is it?

  • Bug fix (non-breaking change which fixes an issue)

I'm not 100% sure if it's a breaking change? If it is, it's only breaking things minimally.

The `color` gets inherited from the `body` and doesn't need to be set.
Furthermore, only setting the colour without also setting
the background colour could cause issues for people
needing to change colour settings.
@gemmaleigh gemmaleigh temporarily deployed to govuk-elements-review-pr-482 June 2, 2017 15:35 Inactive
@selfthinker selfthinker mentioned this pull request Jun 2, 2017
@gemmaleigh
Copy link
Contributor

Thanks @selfthinker & @edwardhorsford 👍

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.

3 participants