[PR #82] [MERGED] Fix/main video #141

New issue

Closed

opened 2026-03-04 00:04:05 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 00:04:05 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/YouG-o/YouTube-No-Translation/pull/82
Author: @YouG-o
Created: 8/17/2025
Status: ✅ Merged
Merged: 8/17/2025
Merged by: @YouG-o

Base: main ← Head: fix/MainVideo

📝 Commits (7)

c97d494 added main title to refreshing when switching back to a tab
8fe6f32 fix: ensure main video observer waits for video-id attribute before processing, improving reliability on initial and SPA loads
749c9e6 changelog
34b37a3 fix: add marker to main title when modified to prevent disappearing or duplicate titles on back navigation
9b9c7e5 change: removed page title fallback on main title fetch (not needed)
3bf4670 fix: updated mini player title selector
4932084 feat: use injected page script to get miniplayer video id

📊 Changes

9 files changed (+330 additions, -194 deletions)

View changed files

📝 CHANGELOG.md (+5 -0)
📝 manifests/manifest.chromium.json (+57 -56)
📝 manifests/manifest.edge.json (+57 -56)
📝 manifests/manifest.firefox.json (+20 -19)
📝 manifests/manifest.safari.json (+1 -0)
📝 scripts/bundle-scripts.js (+1 -0)
📝 src/content/observers.ts (+23 -22)
➕ src/content/titles/getIdFromMiniPlayer.js (+119 -0)
📝 src/content/titles/mainTitle.ts (+47 -41)

📄 Description

Fixed main video title & description still being translated. #81
Fixed min player title still being translated #79

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/YouG-o/YouTube-No-Translation/pull/82 **Author:** [@YouG-o](https://github.com/YouG-o) **Created:** 8/17/2025 **Status:** ✅ Merged **Merged:** 8/17/2025 **Merged by:** [@YouG-o](https://github.com/YouG-o) **Base:** `main` ← **Head:** `fix/MainVideo` --- ### 📝 Commits (7) - [`c97d494`](https://github.com/YouG-o/YouTube-No-Translation/commit/c97d4941355b5d4dedd3cbc383b1c97a1b6ce8c9) added main title to refreshing when switching back to a tab - [`8fe6f32`](https://github.com/YouG-o/YouTube-No-Translation/commit/8fe6f32170d094471aa879733b403f7eba1b93f1) fix: ensure main video observer waits for video-id attribute before processing, improving reliability on initial and SPA loads - [`749c9e6`](https://github.com/YouG-o/YouTube-No-Translation/commit/749c9e62584b3295e36977708fe93cda86ead50f) changelog - [`34b37a3`](https://github.com/YouG-o/YouTube-No-Translation/commit/34b37a3f356a2d3e5c3a6c0ae726acb15176260d) fix: add marker to main title when modified to prevent disappearing or duplicate titles on back navigation - [`9b9c7e5`](https://github.com/YouG-o/YouTube-No-Translation/commit/9b9c7e5383eb60933b71ed075204a2b6b423fc0c) change: removed page title fallback on main title fetch (not needed) - [`3bf4670`](https://github.com/YouG-o/YouTube-No-Translation/commit/3bf46707db66ec7e9037e29b2dd857f5833b7e13) fix: updated mini player title selector - [`4932084`](https://github.com/YouG-o/YouTube-No-Translation/commit/4932084dfa907e69b9af143749f723d7d46924a7) feat: use injected page script to get miniplayer video id ### 📊 Changes **9 files changed** (+330 additions, -194 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+5 -0) 📝 `manifests/manifest.chromium.json` (+57 -56) 📝 `manifests/manifest.edge.json` (+57 -56) 📝 `manifests/manifest.firefox.json` (+20 -19) 📝 `manifests/manifest.safari.json` (+1 -0) 📝 `scripts/bundle-scripts.js` (+1 -0) 📝 `src/content/observers.ts` (+23 -22) ➕ `src/content/titles/getIdFromMiniPlayer.js` (+119 -0) 📝 `src/content/titles/mainTitle.ts` (+47 -41) </details> ### 📄 Description Fixed main video title & description still being translated. #81 Fixed min player title still being translated #79 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>