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

Bug 1125971 - Fixing quotes on /fr/privacy/tips/ and removing extra spac... #2681

Merged

Conversation

TheoChevalier
Copy link
Contributor

...e before the closing quotation mark

the are here to remove the extra space between the string and the closing quotation mark. That's a bit of a hack, let me know if you think of something cleaner ;)

@TheoChevalier
Copy link
Contributor Author

@jpetto Mind giving it a quick look when you have the opportunity?
Thanks :)

@alexgibson
Copy link
Member

I've given this a quick check over and looks fine, thanks @TheoChevalier :)

Only nit is the comment hack to remove the extra white space. This seems to work fine, but I think it might confuse people to see it there in the markup. Perhaps a simpler solution might just to be to inline the content between the <p> tags instead of including the line breaks. Still looks relatively tidy this way, but I'm open to other suggestions if you disagree?

@TheoChevalier
Copy link
Contributor Author

Thanks Alex!
I don't mind about comments, so let's inline p tags if that works for you :)
I updated the PR with this change

@alexgibson
Copy link
Member

LGTM, thanks @TheoChevalier r+ 🎱

alexgibson added a commit that referenced this pull request Jan 30, 2015
Bug 1125971 - Fixing quotes on /fr/privacy/tips/ and removing extra spac...
@alexgibson alexgibson merged commit 0f668e9 into mozilla:master Jan 30, 2015
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.

None yet

2 participants