[PR #30] [MERGED] Fix tooltip overflow issues in popup #119

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

📋 Pull Request Information

Original PR: https://github.com/YouG-o/YouTube-No-Translation/pull/30
Author: @therichkid
Created: 6/2/2025
Status: Merged
Merged: 6/2/2025
Merged by: @YouG-o

Base: mainHead: tooltip-overflow


📝 Commits (1)

  • 5e66e64 Fix tooltip overflow issues

📊 Changes

2 files changed (+18 additions, -6 deletions)

View changed files

📝 src/popup/popup.html (+6 -6)
📝 src/popup/popup.ts (+12 -0)

📄 Description

I saw that the tooltip for the audio tracks and descriptions overflows on the right side of the extension settings menu. This creates the horizontal scroolbar and without scrolling not the whole text is visible. I created a TypeScript solution that changes the left margin if the right side of the tooltip is bigger than the right side of the document body.

Also unified the capitalization of the tooltip text on one occasion.

Good work on the plugin which I use on all my devices! Wanted to help out with this small PR.


🔄 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/30 **Author:** [@therichkid](https://github.com/therichkid) **Created:** 6/2/2025 **Status:** ✅ Merged **Merged:** 6/2/2025 **Merged by:** [@YouG-o](https://github.com/YouG-o) **Base:** `main` ← **Head:** `tooltip-overflow` --- ### 📝 Commits (1) - [`5e66e64`](https://github.com/YouG-o/YouTube-No-Translation/commit/5e66e642f34861384a14635f5ebedec5e379033e) Fix tooltip overflow issues ### 📊 Changes **2 files changed** (+18 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `src/popup/popup.html` (+6 -6) 📝 `src/popup/popup.ts` (+12 -0) </details> ### 📄 Description I saw that the tooltip for the audio tracks and descriptions overflows on the right side of the extension settings menu. This creates the horizontal scroolbar and without scrolling not the whole text is visible. I created a TypeScript solution that changes the left margin if the right side of the tooltip is bigger than the right side of the document body. Also unified the capitalization of the tooltip text on one occasion. Good work on the plugin which I use on all my devices! Wanted to help out with this small PR. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-04 00:03:58 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/YouTube-No-Translation#119
No description provided.