Skip to content

feat: dynamically layout bilingual ASS subtitles#17

Merged
Mapleeeeeeeeeee merged 1 commit into
mainfrom
feat/dynamic-ass-subtitle-layout
Jul 3, 2026
Merged

feat: dynamically layout bilingual ASS subtitles#17
Mapleeeeeeeeeee merged 1 commit into
mainfrom
feat/dynamic-ass-subtitle-layout

Conversation

@Mapleeeeeeeeeee

Copy link
Copy Markdown
Owner

Summary

  • Add dynamic bilingual ASS layout so translated and original subtitle lines stay grouped
  • Wrap source/translated text with top-anchored ASS positions to avoid multiline English pushing upward
  • Add unit coverage for 1, 2, and 3-line original subtitle cases plus burn-in integration coverage

Verification

  • uv run ruff check src/ tests/
  • uv run ruff format --check src/ tests/
  • uv run mypy src/
  • uv run pytest tests/unit -m unit --cov=bilingualsub --cov-report=xml --cov-fail-under=60
  • uv run pytest tests/integration -m integration --cov=bilingualsub --cov-report=xml --cov-fail-under=30
  • uv run pytest tests/e2e -m e2e --cov-fail-under=0

@Mapleeeeeeeeeee Mapleeeeeeeeeee merged commit 180820d into main Jul 3, 2026
7 of 8 checks passed
@Mapleeeeeeeeeee Mapleeeeeeeeeee deleted the feat/dynamic-ass-subtitle-layout branch July 3, 2026 09:38
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.

1 participant