Helper method te() should return an HtmlString and should always escape every parameters

[pgrenaud]

We should avoid using {!! te('key') !!} everywhere and instead always return an HtmlString. We should also always escape all parameters passed in the $replace argument (second argument of the get() method) instead of doing {!! te('key', ['a' => e($input)]) !!} everywhere.

[xel1045]

Indeed, {{ te('key') }} should be used and it's already the supported:

laravel-translation-editor/src/TranslationEditor.php

Line 101 in 584e302

public function getEditor($key, array $replace = [], $locale = null)

For the second feature request (i.e. escaping $replace), I don't think it's a good idea. Since we're only forwarding translations to Laravel's translation getter, I think it's better if we keep trans() and te() aligned.

[pgrenaud]

As discussed, getEditor() does return an HtmlString. But, this method is only called when the editor is enabled. When the editor is disabled, getTranslation() is directly called and this method does not return an HtmlString. You did propose to build a blade component to fix this specific issue, but I do not remember exactly how this will resolve it.