Skip to content

Fix anchor text not translated in ie8 #130

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

Merged
merged 1 commit into from
Dec 8, 2014

Conversation

maxpeterson
Copy link
Contributor

Related #49 that I believe is caused by angular/angular.js#1949 / angular/angular.js#2013

This updates the code to remove the <!--IE fix--> comment if it is found at the end of the string being translated.

This would not fix a string containing a link such as <p translate>This <a>link</a> would not get fixed</p>.

Related rubenv#49 that is caused by
angular/angular.js/#1949 / angular/angular.js/#2013
@maxpeterson
Copy link
Contributor Author

@rubenv does this seem like an acceptable fix for #49 ?

@rubenv rubenv modified the milestone: 2.0 Dec 8, 2014
@rubenv rubenv merged commit 10dcb27 into rubenv:master Dec 8, 2014
@rubenv
Copy link
Owner

rubenv commented Dec 8, 2014

Excellent fix, works as it should, so merged.

Thanks a lot!

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.

2 participants