retro: record PR for issue #20 (already merged to dev) - #21

Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom
dev
Closed

retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom
dev

Conversation

@Kotmin

@KotminKotmin commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Record-only PR

These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.

Closes-by-reference: #20

Commits

  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes

Root cause summary

  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).

Going forward, changes to dev in this repo go through a PR instead of a direct push.

@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin

Kotmin commented Aug 16, 2026

Copy link
Copy Markdown
OwnerAuthor

Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.

@KotminKotmin closed this Aug 16, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
retro: record PR for issue #20 (already merged to dev) - #21
Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Closed
retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Conversation
@Kotmin
@KotminKotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

Owner
Record-only PR
These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.
Closes-by-reference: #20
Commits
  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes
Root cause summary
  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).
Going forward, changes to dev in this repo go through a PR instead of a direct push.
@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin
Kotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

OwnerAuthor
Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.
@KotminKotmin
closed this
Aug 16, 2026
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
retro: record PR for issue #20 (already merged to dev) - #21
Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Closed
retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Conversation
@Kotmin
@KotminKotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

Owner
Record-only PR
These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.
Closes-by-reference: #20
Commits
  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes
Root cause summary
  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).
Going forward, changes to dev in this repo go through a PR instead of a direct push.
@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin
Kotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

OwnerAuthor
Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.
@KotminKotmin
closed this
Aug 16, 2026
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
retro: record PR for issue #20 (already merged to dev) - #21
Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Closed
retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Conversation
@Kotmin
@KotminKotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

Owner
Record-only PR
These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.
Closes-by-reference: #20
Commits
  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes
Root cause summary
  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).
Going forward, changes to dev in this repo go through a PR instead of a direct push.
@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin
Kotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

OwnerAuthor
Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.
@KotminKotmin
closed this
Aug 16, 2026
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
retro: record PR for issue #20 (already merged to dev) - #21
Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Closed
retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Conversation
@Kotmin
@KotminKotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

Owner
Record-only PR
These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.
Closes-by-reference: #20
Commits
  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes
Root cause summary
  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).
Going forward, changes to dev in this repo go through a PR instead of a direct push.
@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin
Kotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

OwnerAuthor
Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.
@KotminKotmin
closed this
Aug 16, 2026
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
retro: record PR for issue #20 (already merged to dev) - #21
Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Closed
retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Conversation
@Kotmin
@KotminKotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

Owner
Record-only PR
These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.
Closes-by-reference: #20
Commits
  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes
Root cause summary
  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).
Going forward, changes to dev in this repo go through a PR instead of a direct push.
@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin
Kotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

OwnerAuthor
Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.
@KotminKotmin
closed this
Aug 16, 2026
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
retro: record PR for issue #20 (already merged to dev) - #21
Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Closed
retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Conversation
@Kotmin
@KotminKotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

Owner
Record-only PR
These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.
Closes-by-reference: #20
Commits
  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes
Root cause summary
  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).
Going forward, changes to dev in this repo go through a PR instead of a direct push.
@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin
Kotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

OwnerAuthor
Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.
@KotminKotmin
closed this
Aug 16, 2026
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
retro: record PR for issue #20 (already merged to dev) - #21
Closed
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Closed
retro: record PR for issue #20 (already merged to dev)#21
Kotmin wants to merge 3 commits into
retro/issue-20-basefrom 
dev
Conversation
@Kotmin
@KotminKotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

Owner
Record-only PR
These three commits already landed on dev via a direct push (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow. This PR exists purely to give them a reviewable diff and a permanent record — it is not meant to be merged; dev already contains this work.
Closes-by-reference: #20
Commits
  • ceb3d02 fix(queue-overlay): match menuitem role on popup item's inner button
  • cbd91d2 feat(playlist): add best-effort Shorts fallback for save-to-playlist trigger
  • 52dfef3 docs(changelog): note queue-overlay and Shorts save-trigger fixes
Root cause summary
  • "Add to queue" broke everywhere (not just short URLs): YouTube moved role="menuitem" off the popup item wrapper onto its inner button/link.
  • "Save to playlist" (Shift+P) no-op'd on Shorts specifically: the watch-page trigger selector doesn't exist on the Shorts player; added a best-effort fallback to the Shorts header "..." menu, flagged as not yet live-confirmed (docs/ai/questions-for-K.md Q17).
Going forward, changes to dev in this repo go through a PR instead of a direct push.
@Kotmin
fix(queue-overlay): match menuitem role on popup item's inner button
ceb3d02
YouTube moved role="menuitem" off yt-list-item-view-model onto its
inner button/anchor (wrapper is now role="presentation"), so the
queue popup opened but the "Add to queue" item was never found,
silently no-opping. Select the descendant instead of the wrapper.
@Kotmin
feat(playlist): add best-effort Shorts fallback for save-to-playlist …
cbd91d2
…trigger
findSaveToPlaylistTrigger only looked for the /watch page's action-row
button, so Ctrl+A Shift+P silently no-op'd on the Shorts standalone player
(issue #20). A captured Shorts-page JSON snapshot confirms YouTube offers
"Save to playlist" there via the header "..." menu, so add
ytd-reel-player-header-renderer ytd-menu-renderer button as a fallback,
derived from JSON renderer names and this codebase's established naming
convention but not yet confirmed against a live DOM.
@Kotmin
docs(changelog): note queue-overlay and Shorts save-trigger fixes
52dfef3
@Kotmin
Kotmin
commented
Aug 16, 2026

Copy link


Copy Markdown

OwnerAuthor
Closing — this PR was opened purely as a reviewable record of commits already merged directly to dev (branch protection was bypassed by an admin-rights account) before the repo switched to a PR-required workflow for dev. No merge needed; dev already contains this work (see #20). Going forward, changes to dev land via PR, not direct push.
@KotminKotmin
closed this
Aug 16, 2026
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

1 participant

@Kotmin