Avoid using HTML tags in translation strings (bp-core/classes/class-bp-admin.php)

[ramiy]

See the attaches patch.

[DJPaul]

I don't like this patch because it splits a paragraph into two sentences. I expect there may be languages where people want to re-order the sentences, or continue to have just one for both.

There's also no benefit to providing a URL for translation as the project hasn't tackled translating its codex (yet).

[ramiy]

@DJPaul

This patch is splitting the original string into two strings. The first string is an informative text where you inform the user about the community upload. The second string is a call-to-action.

But I agree with you that we need to remove the codex link translation string.

---

Old string:

• Your community can now upload cover photos for their user profiles and groups. <a href="%s">Learn more &rarr;</a>

New strings:

• Your community can now upload cover photos for their user profiles and groups.
• Learn more →

[boonebgorges]

I'm going to agree with @DJPaul on this one. "Learn more →" doesn't make much sense without the context of the previous sentence.