Yes, that is possible. Do you know about other places where this change should be done?

Not directly, but I might have a look at it again in the next days to check which strings actually cause problems at the moment (I did not take notes on them the last time).

That would be nice, then I can fix all of them at once.

I just went through all the strings at Weblate and found the following strings/messages which are problematic in my opinion (at least for German translations):

• ContactTab.cpp

  tr("If you have general questions, ideas or just want to talk about ksnip,") + QLatin1String("<br/>") +
    		tr("please join our") + QLatin1String(" <a href=\"http://discord.ksnip.org\">Discord</a> ") +
    		tr("or our") + QLatin1String(" <a href=\"https://web.libera.chat/?channels=#ksnip\">IRC</a> ") +
    		tr("server")	+ QLatin1String(".<br/><br/>")
  

• ContactTab.cpp - tr("Please use ") + QLatin1String("<a href=\"https://github.com/ksnip/ksnip/issues/\">GitHub</a>") + tr(" to report bugs.")

• DonateTab.cpp - tr("Also possible, ") + QLatin1String(" <a href=\"https://github.com/sponsors/DamirPorobic\">") + tr("here") + QLatin1String("</a>.")

• DonateTab.cpp

  tr("If you want to help or just want to appreciate the work being done<br/>"
    				         "by treating developers to a beer or coffee, you can do that") + QLatin1String(" <a href=\"https://www.paypal.me/damirporobic\">") + tr("here")
  

• HandleUploadResultOperation.cpp - tr("Upload script ") + mConfig->uploadScriptPath() + tr(" finished successfully.")

• HandleUploadResultOperation.cpp - tr("Uploaded to") + QLatin1String(" ") + url

• RenameOperation.cpp - tr("Successfully renamed image to ") + newFilename

• RenameOperation.cpp - tr("Failed to rename image to ") + newFilename

There are some strings with problems which might not be related to this issue directly, but were not always clear to me and might be useful to be discussed:

• DonateTab.cpp - tr("Become a GitHub Sponsor") + QLatin1String("?</b>") - It might be relevant that this is a question.
• MainToolBar.cpp - tr("s") - Unclear without context, although there is a comment in this case at Weblate explaining it.
• ActionSettingTab.cpp - tr("s") - Unclear without context as in the previous case.
• SaveOperation.cpp - tr("Saved to") - No way to include path. Is this correct?
• SaveOperation.cpp - tr("Failed to save image to") - No way to include path. Is this correct?

Would you expect line breaks to be part the translation string when the line break is happening in the sentence?

And thanks for collecting those, I'll try to fix them.

Would you expect line breaks to be part the translation string when the line break is happening in the sentence?

If you mean simple newlines (\n), then I would say yes for now, as there already are strings with sentences split across multiple lines. Although I would not argue against automatic line breaks inside the strings if this is possible for all languages, as this would not require the translators to care about such display details - nevertheless it should be okay for now to insert them manually.

Also those <br/>.

I'll check for an automatic way to handle this but if I recall correctly that wasn't going for some reason.

If those <br/> are just used as the \n equivalent in HTML strings, the same should apply as in my previous comment. For other cases: Do you have an example of what you mean with it?

That's exactly what I mean. In the about dialog we have html tagged text and all other places should be using the newline character.

I think I was able to fix all entries you had there. Also the one with "s" should have now a comment, never used it but the docu says I can add one, which I did.