Skip to content

Reverse Translation - #510

Closed
iHad168 wants to merge 1 commit into
you-apps:masterfrom
iHad168:master
Closed

Reverse Translation#510
iHad168 wants to merge 1 commit into
you-apps:masterfrom
iHad168:master

Conversation

@iHad168

@iHad168 iHad168 commented Jul 16, 2025

Copy link
Copy Markdown
Contributor
  • Added reverse translation result function (used as a contrast between source and target meaning deviations)
  • List multiple translators at the same time Translate If
  • Fix Apertium DeepL Glosbe Lingva Mozhi MinT Yandex Traditional/Simplified Chinese LanguageCode
  • When source is "auto" can switch source and target Language
圖片

Added reverse translation result function (used as a contrast between source and target meaning deviations)
List multiple translators at the same time Translate If
Fix Apertium DeepL Glosbe Lingva Mozhi MinT Yandex Traditional Chinese LanguageCode

@Bnyro Bnyro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this PR!

Please split your changes into three different pull requests (using different branches):

  • One for the reverse translation function
  • One for fixing the chinese language codes
  • One for listing multiple translators at once

This makes it easier to review the changes, and merge them separately, as they're not related to each other.

As a side note: As we display all simultaneous translation results at the same time now, we don't even have to display bottom bar that allows to select one of the engines anymore, right? We could just allow users to click on the returned translation to select a new engine.

@iHad168

iHad168 commented Jul 19, 2025

Copy link
Copy Markdown
Contributor Author

#513
#514
#515
#516

@Bnyro

Bnyro commented Jul 19, 2025

Copy link
Copy Markdown
Member

Thank you! I'll review/merge them in the next few days.

@Bnyro Bnyro closed this Jul 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants