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

[#751] Fix apostrophe tags in help pages #753

Merged
merged 4 commits into from Feb 2, 2021

Conversation

mdeuk
Copy link
Collaborator

@mdeuk mdeuk commented Jan 25, 2021

Relevant issue(s)

Fixes #751

What does this do?

This PR swaps out the apostrophe character that we've used in some help pages.

Why was this needed?

We've inconsistently used both ' and within help documents, which creates a slight semantic issue and makes the code slightly clunky to work with.

Implementation notes

This is a straight swap of characters in almost all cases - taking care not to mess with legitimate instances of the ' character within links.

Screenshots

N/A

Notes to reviewer

I have intentionally skipped unhappy.html.erb since changes to this file will likely be committed elsewhere.

This replaces a previous PR which contained a glitch that had been pushed in error.

Minor change to multiple files, correcting the apostrophe tag from ' to ’
A minor correction to the apostrophe tag we use in multiple paragraphs.

There is no other change to the substance of the Privacy Notice.
Minor corrections to the apostrophe used within multiple files
A minor change to correct the apostrophe tags used within the House Rules to provide consistency.

There are no other changes.
@mdeuk mdeuk marked this pull request as ready for review January 25, 2021 09:22
@mdeuk mdeuk changed the title Fix apostrophe tags in help pages [#751] Fix apostrophe tags in help pages Jan 25, 2021
Copy link
Member

@garethrees garethrees left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Epic! Thanks!

I haven't reviewed this line by line, but all looks great in principle and I've checked that all the help pages still render.

@mysociety-pusher mysociety-pusher merged commit 7f17761 into mysociety:master Feb 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correct inconsistent use of apostrophe characters
3 participants