Bug 512355

Summary:	Some effects/composite descriptions, option names and option tooltips show always in English.
Product:	[Applications] kdenlive	Reporter:	Gabriel Gazzán <gabcorreo>
Component:	Translation	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	REOPENED ---
Severity:	normal	CC:	berndmj
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Other
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Gabriel Gazzán 2025-11-19 22:17:23 UTC

Hi,
This comes from discussion at:
https://invent.kde.org/multimedia/kdenlive/-/commit/54b8afd8e1c208f27267e345c61c89cb7a827629#note_1349712

It seems that strings that contain <![CDATA[ at the beginning are not being shown in their translated form, or at least is the conclusion I've come to.

Example of the problem can be seen in the following effects, among many others:
TAP Equalizer - effect description, several option names ("Band n Freq")
TAP Fractal Doubler - effect description, options numeric fields tooltips.

It happens in some video effects also, like:
Chroma Hold - effect description*, "Similarity" option tooltip. 
  *the description of this effect doesn't start with <![CDATA[ though.

Comment 1 Jean-Baptiste Mardelle 2025-11-20 12:30:16 UTC

Looking at the translation file, I now realize that the <![CDATA[..]]> tags are included in the string to be translated.
https://invent.kde.org/multimedia/kdenlive/-/blob/master/po/es/kdenlive.po?ref_type=heads#L7643

However, when we try to translate these tags do not appear. This should be stripped from the source string, as it is only used as an xml storage conveninence. I will check if we can update the Messages pipeline to remove the CDATA tags.

Comment 2 Gabriel Gazzán 2025-11-20 14:26:40 UTC

ok, that may be it.
thanks!

Comment 3 Bug Janitor Service 2025-11-20 15:13:50 UTC

A possibly relevant merge request was started @ https://invent.kde.org/sdk/kde-dev-scripts/-/merge_requests/43

Comment 4 Jean-Baptiste Mardelle 2025-11-25 04:06:29 UTC

Git commit 02fdbf1848a883fd6f32da4885c5e3aad45af795 by Jean-Baptiste Mardelle.
Committed on 25/11/2025 at 04:06.
Pushed by mardelle into branch 'release/25.12'.

Correctly extract strings with CDATA, now that extractrc supports it

M  +1    -1    Messages.sh

https://invent.kde.org/multimedia/kdenlive/-/commit/02fdbf1848a883fd6f32da4885c5e3aad45af795

Comment 5 Jean-Baptiste Mardelle 2025-11-25 04:12:44 UTC

Ok, so everything is now in place to remove the CDATA strings from strings to be translated. It means unfortunately that all strings with <![CDATA[ will be changed, but once translated they should now appear correctly

Please check and let me know if this solves the problem.

Comment 6 Jean-Baptiste Mardelle 2025-11-26 07:25:48 UTC

Strings in git master now appear without the CDATA in po files. Could you check if things work as expected once translated? If that works, I will also apply the fix to 25.12 branch

Comment 7 Gabriel Gazzán 2025-11-27 13:24:12 UTC

Great, thanks!
I'm on a trip right now, on Saturday when I get back home I'll test it. 💪

Comment 8 Gabriel Gazzán 2025-11-30 04:23:03 UTC

All seems to be working great, now.
Thanks!! 😊

Comment 9 Bernd 2025-12-01 00:19:10 UTC

(In reply to Gabriel Gazzán from comment #8)
> All seems to be working great, now.
> Thanks!! 😊

Closing it as RESOLVED - FIXED

Comment 10 Gabriel Gazzán 2025-12-19 01:42:17 UTC

For some reason this one is still present in the 25.12.0 release.
(in the last nightly [11757] they show translated, though)