⚡️Improve AI translations formatting #662
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AntoLC
force-pushed
the
refacto/unlossy-ai-translation-the-return
branch
from
0bd97a5 to
bf22553
Compare
AntoLC
changed the title
AntoLC
force-pushed
the
refacto/unlossy-ai-translation-the-return
branch
from
bf22553 to
8133d29
Compare
lunika
approved these changes
Mar 12, 2025
AntoLC
force-pushed
the
refacto/unlossy-ai-translation-the-return
branch
from
3e8a2cf to
e8161b9
Compare
virgile-dev
approved these changes
Mar 13, 2025
We loose the custom blocks yes, background and usual formatting are normally not lost anymore, but it can happen yes. |
The ai translation were quite lossy about formatting. Colors, background, breaklines, table sizes were lost in the translation. We improve the AI translation request to keep the formatting as close as possible by using html instead of markdown.
The beautify action add emojis to the important parts of the text and add formatting to the text to make it more readable.
The emojify action add emojis to the important parts of the text.
Use the blockquote tag for quote block instead of a paragraph tag.
AntoLC
force-pushed
the
refacto/unlossy-ai-translation-the-return
branch
from
e8161b9 to
77467f6
Compare
|
J'ai review aussi c'est top. Passer par l'export html me semble bien, au pire on aura des artifacts de balise html si jamais le modèle se plante. |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
To translate with the AI we are using lossy functions
blocksToMarkdownLossy/tryParseMarkdownToBlocks, it transforms the content editor to markdown. The problem is this transpilation is very lossy, we loose color, background, table size, line break etc...Proposal
We will use html instead of markdown to do the translation, html can carry much more information that markdown about the content style.
Demo
What we have now with markdown :
scrnli_Ug6vr23om8ZN8E.webm
With preserved formatting :
scrnli_aTe1Ed2p695SF2.webm
Feature Beautify
scrnli_v7YQ56N18Q3O5n.webm