Skip to content

Add bilingual option (show both original text and translated text at the same time) -- for language learners#852

Open
rholop wants to merge 15 commits into
bbepis:masterfrom
rholop:master
Open

Add bilingual option (show both original text and translated text at the same time) -- for language learners#852
rholop wants to merge 15 commits into
bbepis:masterfrom
rholop:master

Conversation

@rholop

@rholop rholop commented Jun 14, 2026

Copy link
Copy Markdown

No description provided.

claude and others added 15 commits June 11, 2026 17:43
Add bilingual mode to display original and translated text
Merge pull request #1 from rholop/claude/confident-hopper-gxgyvx
Add release CI workflow and document dual-subtitles bilingual mode
Fix BilingualFormat: unescape \n in config so it renders as a newline
…ore RuntimeHooker

The PreBuild XCOPY in XUnity.RuntimeHooker.csproj depends on the output of
XUnity.RuntimeHooker.Trampolines, but no project reference enforced that
build order, causing intermittent MSB3073 failures under msbuild /m.
Remove the fork-specific "About this fork" section and instead
introduce bilingual mode as a regular feature in the Introduction,
pointing to the existing BilingualMode/BilingualFormat config docs.

@ManlyMarco ManlyMarco left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please revert changes unrelated to bilingual mode, e.g. the workflow, build instructions and csproj changes. I think the introduction section change is also unnecessary and should be removed.

The code changes themselves look correct.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a bilingual/dual-subtitles mode to the AutoTranslator plugin so language learners can see original + translated text simultaneously, plus updates docs and adds a CI workflow to build release packages.

Changes:

  • Introduces BilingualMode and BilingualFormat configuration options (and forces UI resizing when enabled).
  • Composes and sets bilingual UI text in the core translation flow, and avoids re-processing known bilingual composites.
  • Documents bilingual mode + adds “Building from Source” documentation and a GitHub Actions workflow for Release builds/artifacts.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/XUnity.RuntimeHooker/XUnity.RuntimeHooker.csproj Adds a project reference to trampolines (non-output reference) to support build/link requirements.
src/XUnity.AutoTranslator.Plugin.Core/Configuration/Settings.cs Adds config surface for bilingual mode/format and forces UI resizing when enabled.
src/XUnity.AutoTranslator.Plugin.Core/AutoTranslationPlugin.cs Implements bilingual string composition + composite detection in translation paths and toggle flow.
README.md Documents bilingual mode and adds source build/CI packaging documentation.
.github/workflows/build-release.yml Adds Windows CI to build Release and publish dist/*.zip artifacts and tagged releases.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/XUnity.AutoTranslator.Plugin.Core/AutoTranslationPlugin.cs
Comment thread src/XUnity.AutoTranslator.Plugin.Core/AutoTranslationPlugin.cs
Comment thread .github/workflows/build-release.yml
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.

4 participants