Twenty Nineteen: Avoid html code in translatable strings

[Presskopp]

As always only the translatable part should appear for translators to avoid issues.

I came across this one:

<span class="meta-nav">Published in</span><br><span class="post-title">%title</span>

[pratikkry]

Github ​725

[mukesh27]

Patch.

[laurelfulford]

Thanks for the report, @Presskopp!

It looks like there's been quite a bit of discussion about the best approach for this issue, shared in Twenty Nineteen GitHub repo, on top of ​https://github.com/WordPress/twentynineteen/issues/725 shared above:

• ​https://github.com/WordPress/twentynineteen/pull/274 (looks like the bulk of the conversation happens here)
• ​https://github.com/WordPress/twentynineteen/pull/733
• ​https://github.com/WordPress/twentynineteen/pull/734

@kjellr or @allancole -- I'm definitely not up to date where this ended up in Twenty Nineteen. Is there a particular direction that was decided on for HTML in strings marked for translation? From my read it sounds like it may make sense to keep the HTML in some cases, but I could be wrong!

[laurelfulford]

It seems like this one could be a bit too involved for 5.0.3, so punting to 5.1.

[kjellr]

From my read it sounds like it may make sense to keep the HTML in some cases, but I could be wrong!

That's my understanding here too, but I'd defer to the i18n experts.

Regarding @mukesh27's patch above, @grapplerulrich noted that this may not work because some languages may need to have this string rearranged to say "Title published in".

[laurelfulford]

Punting this issue for now, as we figure out exactly what strings need to be updated.

[SergeyBiryukov]

Just noting the string in question is not much different from similar "Published in" strings in Twenty Fourteen, Fifteen, and Sixteen, so the issue is not unique to Twenty Nineteen.

[desrosj]

This still needs work and has not been touched since being punted to 5.2. Moving to Future Release until someone is able to work on this.