broulik created this revision. broulik added reviewers: Plasma, fvogt. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision.
REVISION SUMMARY `chrome://` and `about:` pages are typically local and browser-specific, and `blob:` pages cannot be shared altogether. Only `targetUrlPatterns` but not `documentUrlPatterns` is set so you can still send links from your "New Tab" or history page, evne though that'll mean you could try sharing the page itself which will fail. TEST PLAN Should we include "ftp"? - No longer get a defunct "Open on $phone" entry for dynamic video streams behind a `blob:` URL - No longer get a defunct "Open on $device" entry for many links in browser settings. Ones that only do JS and have an empty target still get it, unfortunately. - Can still share links on my "New Tab" and browser history pages REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D28939 AFFECTED FILES extension/extension-kdeconnect.js To: broulik, #plasma, fvogt Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart