fix(directive): translate by attribute (not binding) with translation key as value #1268
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix for angular 9 & 10
This PR fixes the behaiviour of
TranslateDirective
when it is used as a simple attributetranslate
(not attribute binding[translate]
) with value and translated element already contains some text. Example:Currently (without this fix) this code will result in text translated twice. If value for
'TEST'
in english is "This is a test", after directive is executed, value of #attributeWithValue will be "This is a testThis is a test".There was no such problem while using ng-translate with Angular 8. But since Angular 9 it appeared.
This PR also contains some refactorings.