D24828: KRun: don't override user preferred app when opening local *.*html and co. files
ahmadsamir added a subscriber: ngraham. ahmadsamir added a comment. @ngraham: I do know how to land a diff :) REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24828 To: ahmadsamir, #frameworks, dfaure Cc: ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns
D24828: KRun: don't override user preferred app when opening local *.*html and co. files
This revision was automatically updated to reflect the committed changes. Closed by commit R241:afb2f80cbbe1: KRun: dont override user preferred app when opening local *.*html and co. files (authored by ahmadsamir, committed by ngraham). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24828?vs=68429=70784 REVISION DETAIL https://phabricator.kde.org/D24828 AFFECTED FILES src/widgets/krun.cpp To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D24828: KRun: don't override user preferred app when opening local *.*html and co. files
ngraham edited the summary of this revision. REPOSITORY R241 KIO BRANCH ahmad/separate-browser-html (branched from master) REVISION DETAIL https://phabricator.kde.org/D24828 To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D24828: KRun: don't override user preferred app when opening local *.*html and co. files
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. This makes perfect sense to me, thanks for the detailed analysis and the fix. Remember to update the version number in the FIXED-IN line (sorry for the delay in the review). REPOSITORY R241 KIO BRANCH ahmad/separate-browser-html (branched from master) REVISION DETAIL https://phabricator.kde.org/D24828 To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D24828: KRun: don't override user preferred app when opening local *.*html and co. files
ahmadsamir added a comment. Ping. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24828 To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D24828: KRun: don't override user preferred app when opening local *.*html and co. files
ahmadsamir created this revision. ahmadsamir added reviewers: Frameworks, dfaure. Herald added a project: Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY KRun unconditionally opened files with mimetype text/html (and co.) with the browser application set in kdeglobals, BrowserApplication=, and if it's not set, it fells back to mimeapps.list (x-scheme-handler/http*). IMHO, there are two distinctive use cases here, opening http*:// links with a web browser (which is what the user wants most of the time); and opening local '.*html' files on the user's machine, usually with the intention to edit them; the two cases can/should be mutually exclusive. This gives the choice back to the user, as he can set his preferred app to open local .html files, be it a web browser or a text editor. This won't impact new user accounts because most browsers .desktop files have Mimetype entries that cover most of the .html and co. types. So it'll either be the default app already or the user can select his favourite browser from the open-with list. Thanks to broulik for pin-pointing the location in the code that caused this behaviour first. BUG: 399020 FIXED-IN: 5.64 TEST PLAN - Set some web browser as the default in the component chooser kcm - Open Dolphin and using any .html file set the default app to, say, kwrite - Try to open a local .html file, notice that it opens in the web browser you set - Apply the patch, then try again, it should be opened with kwrite REPOSITORY R241 KIO BRANCH ahmad/separate-browser-html (branched from master) REVISION DETAIL https://phabricator.kde.org/D24828 AFFECTED FILES src/widgets/krun.cpp To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns