D18762: Add support for portals in KRun

2019-02-21 Thread Denis Karpovskii
GeeXT added a comment. @apol I didn't applied for KDE Developer Account yet (I'm new here) so I can't, sorry. REPOSITORY R241 KIO BRANCH flatpak_open_url (branched from master) REVISION DETAIL https://phabricator.kde.org/D18762 To: GeeXT, jgrulich, apol, dfaure, #frameworks Cc: kde-f

D18762: Add support for portals in KRun

2019-02-20 Thread Denis Karpovskii
GeeXT updated this revision to Diff 52163. GeeXT added a comment. Swap the conditions REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18762?vs=51047&id=52163 BRANCH flatpak_open_url (branched from master) REVISION DETAIL https://phabricator.kde.org/D18762

D18762: Add support for portals in KRun

2019-02-20 Thread Denis Karpovskii
GeeXT marked an inline comment as done. REPOSITORY R241 KIO BRANCH flatpak_open_url (branched from master) REVISION DETAIL https://phabricator.kde.org/D18762 To: GeeXT, jgrulich, apol, dfaure, #frameworks Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18673: Add symlinks for flatpak

2019-02-16 Thread Denis Karpovskii
GeeXT added a reviewer: Frameworks. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18673 To: GeeXT, #frameworks Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18762: Add support for portals in KRun

2019-02-16 Thread Denis Karpovskii
GeeXT added a comment. Ping? Btw, I couldn't find any usages of setEnableExternalBrowser in BrowserRun or KonqRun but there is an usage after instantiating an object of KonqRun (in KonqMainWindow::openUrl). It's strange that a caller must call the function itself or it will crash... REPOS

D18762: Add support for portals in KRun

2019-02-06 Thread Denis Karpovskii
GeeXT updated this revision to Diff 51047. GeeXT added a comment. Add check for external browser enablement status REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18762?vs=51017&id=51047 BRANCH flatpak_open_url (branched from master) REVISION DETAIL https:

D18762: Add support for portals in KRun

2019-02-05 Thread Denis Karpovskii
GeeXT updated this revision to Diff 51017. GeeXT added a comment. The result should be negated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18762?vs=50972&id=51017 BRANCH flatpak_open_url (branched from master) REVISION DETAIL https://phabricator.kde.or

D18762: Add support for portals in KRun

2019-02-05 Thread Denis Karpovskii
GeeXT updated this revision to Diff 50972. GeeXT added a comment. Restored the accidentally deleted empty line... REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18762?vs=50968&id=50972 BRANCH flatpak_open_url (branched from master) REVISION DETAIL https:/

D18762: Add support for portals in KRun

2019-02-05 Thread Denis Karpovskii
GeeXT created this revision. GeeXT added a reviewer: jgrulich. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. GeeXT requested review of this revision. REVISION SUMMARY Currently KRun in Flatpak/Snap doesn't support portals so it can't launch an application

D18673: Add symlinks for flatpak

2019-02-02 Thread Denis Karpovskii
GeeXT created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. GeeXT requested review of this revision. REVISION SUMMARY This patch creates symlinks (for every application I've found on Flathub and for which Breeze icon theme has an icon) to l