Deepl sometimes adds tag that breakes CKEditor

Created on 22 January 2025, 2 months ago

Problem/Motivation

During the translation of the fullhtml text field sometimes Deepl adds opened tag like on the image below.
Since this tag is not closed, CKeditor breakes and does not show the rest of the translated content.

Steps to reproduce

Add long string text field with full html.
Translate content.
Check translation job item.

Proposed resolution

Do not know the source of the problem that's why cant offer resolution for now.

Remaining tasks

User interface changes

API changes

Data model changes

πŸ› Bug report
Status

Active

Version

2.2

Component

Code

Created by

πŸ‡ΊπŸ‡¦Ukraine anpolimus Kyiv

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @anpolimus
  • πŸ‡©πŸ‡ͺGermany SteffenR Germany

    @anpolimus
    Can you show me the configuration of your translator. While looking at the source code in the screenshot, the whole translation seems to be messed up.

    The configuration should look as following:

    We are using the module also with Full-HTML on larger sites and never had those issues.

    To reproduce i need the following things:

    • configuration of the deepL translator
    • HTML source/ text, which should be translated
  • πŸ‡ΊπŸ‡¦Ukraine anpolimus Kyiv

    Hi, @steffenr. Thank you for your answer.
    Here is my config page. It is quite simple one.
    I see that mine differs from yours by different value of Tag Handling field.
    let me try html value like your one is set.

  • πŸ‡©πŸ‡ͺGermany SteffenR Germany

    Since you are translating content, which includes HTML tags, you must set tag_handling to HTML.
    Otherwise you run into problems, like you are showing in the issue.

    More information on tag_handling can also be found in the DeepL API documentation: https://developers.deepl.com/docs/xml-and-html-handling/html

  • πŸ‡ΊπŸ‡¦Ukraine anpolimus Kyiv

    It seems that using html value for Tag handling fixes this issue.
    I think we have to close this issue.
    Do you want to make html value as default for Deepl configuration?

  • πŸ‡©πŸ‡ͺGermany SteffenR Germany

    I’ll close the ticket, since it was a configuration issue.
    The default should be kept to be β€žoffβ€œ and users can enable the tag_handling as desired.

  • πŸ‡ΊπŸ‡¦Ukraine anpolimus Kyiv

    Ok, thank you for your support.

Production build 0.71.5 2024