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

Allow users to right align table cells #2574

Merged
merged 2 commits into from May 4, 2016
Merged

Allow users to right align table cells #2574

merged 2 commits into from May 4, 2016

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented May 3, 2016

For columns containing numbers it’s easier to scan the list if they are right aligned (eg if every number is two decimal places).

https://trello.com/c/6ld11Eck/346-right-alignment-in-tables-in-govspeak-medium
https://govuk.zendesk.com/agent/tickets/1271189

(Copy has been ran past Liz L)

Guidance

screen shot 2016-05-03 at 14 29 30

Output

screen shot 2016-05-03 at 14 29 16

fofr added 2 commits May 3, 2016
For columns containing numbers it’s easier to scan the list if they are
right aligned (eg every number is two decimal places).
* Increase minimum Kramdown version to 1.10.0
* Include support for right aligned columns in table
@boffbowsh
Copy link
Contributor

@boffbowsh boffbowsh commented May 3, 2016

👍

1 similar comment
@tvararu
Copy link
Contributor

@tvararu tvararu commented May 4, 2016

👍

@fofr fofr merged commit 47bf7da into master May 4, 2016
1 check passed
1 check passed
default Build #8237 succeeded on Jenkins
Details
@fofr fofr deleted the table-columns-help branch May 4, 2016
fofr added a commit that referenced this pull request May 5, 2016
The kramdown upgrading is causing errors to appear in Errbit. Needs
further investigating. Pulling this change.

This reverts commit 47bf7da, reversing
changes made to cccc6b4.
fofr added a commit that referenced this pull request May 5, 2016
Revert "Merge pull request #2574 from alphagov/table-columns-help"
@fofr fofr mentioned this pull request May 9, 2016
1 of 1 task complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.