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 break up i18n string #6225

Merged
merged 3 commits into from
Dec 12, 2014
Merged

Don't break up i18n string #6225

merged 3 commits into from
Dec 12, 2014

Conversation

singingwolfboy
Copy link
Contributor

No description provided.

@@ -18,7 +18,9 @@ <h1 class="page-header">${_("Studio Account Activation")}</h1>
<h1 class="title">${_('Your account activation is invalid')}</h1>
<div class="copy">
<p>${_("We're sorry. Something went wrong with your activation. Check to make sure the URL you went to was correct &mdash; e-mail programs will sometimes split it into two lines.")}</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Is use of the &mdash; required here?

@sarina
Copy link
Contributor

sarina commented Dec 12, 2014

Thanks for catching. Sorry for catching more. 👍 once comments addressed.

@singingwolfboy singingwolfboy force-pushed the studio-i18n-improvements branch from fc33632 to 71a74c4 Compare December 12, 2014 14:43
@cahrens
Copy link

cahrens commented Dec 12, 2014

I did not check out and test this, but the change looks reasonable. 👍

singingwolfboy added a commit that referenced this pull request Dec 12, 2014
@singingwolfboy singingwolfboy merged commit e53263d into master Dec 12, 2014
@singingwolfboy singingwolfboy deleted the studio-i18n-improvements branch December 12, 2014 16:10
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