Skip to content

Commit 829efc7

Browse files
committed
fix some issues with some urls not working
1 parent 2a373a2 commit 829efc7

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

background.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,10 @@ browser = browser || chrome;
1212
const menus = browser.menus || browser.contextMenus;
1313

1414
function createMpvSchemeURI(url) {
15-
return `mpv://watch#${url.replace(/'/g, "%27")}`;
15+
const decodedURL = decodeURI(url);
16+
const encodedURL = encodeURI(decodedURL).replace(/'/g, "%27");
17+
console.debug({url, encodedURL, same: decodedURL === decodeURI(encodedURL)});
18+
return `mpv://watch#${encodedURL}`;
1619
}
1720

1821
async function openInMpv(url) {

open-in-mpv.plugin.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,10 @@ const settings = {
1515
};
1616

1717
function createMpvSchemeURI(url) {
18-
return `mpv://watch#${url.replace(/'/g, "%27")}`;
18+
const decodedURL = decodeURI(url);
19+
const encodedURL = encodeURI(decodedURL).replace(/'/g, "%27");
20+
// console.debug({url, encodedURL, same: decodedURL === decodeURI(encodedURL)});
21+
return `mpv://watch#${encodedURL}`;
1922
}
2023

2124
function contextMenuPatch(tree, context) {

0 commit comments

Comments
 (0)