[PR #72] [MERGED] Refact/main description #135

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/YouG-o/YouTube-No-Translation/pull/72
Author: @YouG-o
Created: 7/25/2025
Status: ✅ Merged
Merged: 7/25/2025
Merged by: @YouG-o

Base: main ← Head: refact/mainDescription

📝 Commits (6)

928ed65 refactor: factorize description DOM insertion logic in updateDescriptionElement
936e0b7 refactor: centralize strict description comparison logic
b5b94c1 refactor: removed old cache system for main description, now use the new one (same as search descriptions)
3298090 refactor:
a779d86 refactor: extract timestamp parsing logic to createTimestampFragment for SRP
fccdfec refactor: Create createUrlLink utility for URL anchor generation in description parsing.

📊 Changes

11 files changed (+417 additions, -493 deletions)

View changed files

📝 manifests/manifest.chromium.json (+1 -1)
📝 manifests/manifest.edge.json (+1 -1)
📝 manifests/manifest.firefox.json (+1 -1)
📝 manifests/manifest.safari.json (+1 -1)
📝 scripts/bundle-scripts.js (+1 -1)
📝 src/content/channel/channelPlayer.ts (+1 -1)
➕ src/content/description/MainDescription.ts (+404 -0)
📝 src/content/description/MainDescriptionScript.js (+0 -0)
➖ src/content/description/descriptionIndex.ts (+0 -263)
📝 src/content/index.ts (+2 -2)
📝 src/content/observers.ts (+5 -222)

📄 Description

Refact (SPR & DRY)
Now uses new cache system (same as search descriptions)
Removed old cache

_{🔄 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/72 **Author:** [@YouG-o](https://github.com/YouG-o) **Created:** 7/25/2025 **Status:** ✅ Merged **Merged:** 7/25/2025 **Merged by:** [@YouG-o](https://github.com/YouG-o) **Base:** `main` ← **Head:** `refact/mainDescription` --- ### 📝 Commits (6) - [`928ed65`](https://github.com/YouG-o/YouTube-No-Translation/commit/928ed65c9d1f2c4752379ad735e50696ce0666fa) refactor: factorize description DOM insertion logic in updateDescriptionElement - [`936e0b7`](https://github.com/YouG-o/YouTube-No-Translation/commit/936e0b75e602354915b57133b05f5b0ec576cc90) refactor: centralize strict description comparison logic - [`b5b94c1`](https://github.com/YouG-o/YouTube-No-Translation/commit/b5b94c16a65fc2dd9d69b43c0462585dc9f49dd6) refactor: removed old cache system for main description, now use the new one (same as search descriptions) - [`3298090`](https://github.com/YouG-o/YouTube-No-Translation/commit/3298090af2cfd5206bce044cad2209ca1b1fd2f6) refactor: - [`a779d86`](https://github.com/YouG-o/YouTube-No-Translation/commit/a779d86fe4d2329052e5a7c858756f2e77341053) refactor: extract timestamp parsing logic to createTimestampFragment for SRP - [`fccdfec`](https://github.com/YouG-o/YouTube-No-Translation/commit/fccdfecec73acfb073f3a3197718a996465b3801) refactor: Create createUrlLink utility for URL anchor generation in description parsing. ### 📊 Changes **11 files changed** (+417 additions, -493 deletions) <details> <summary>View changed files</summary> 📝 `manifests/manifest.chromium.json` (+1 -1) 📝 `manifests/manifest.edge.json` (+1 -1) 📝 `manifests/manifest.firefox.json` (+1 -1) 📝 `manifests/manifest.safari.json` (+1 -1) 📝 `scripts/bundle-scripts.js` (+1 -1) 📝 `src/content/channel/channelPlayer.ts` (+1 -1) ➕ `src/content/description/MainDescription.ts` (+404 -0) 📝 `src/content/description/MainDescriptionScript.js` (+0 -0) ➖ `src/content/description/descriptionIndex.ts` (+0 -263) 📝 `src/content/index.ts` (+2 -2) 📝 `src/content/observers.ts` (+5 -222) </details> ### 📄 Description Refact (SPR & DRY) Now uses new cache system (same as search descriptions) Removed old cache --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>