]> cloud.milkyroute.net Git - dolphin.git/commit
Swap "Open in New Window" and "Open in New Tab" in context menus
authorIlya Bizyaev <bizyaev@zoho.com>
Tue, 11 Feb 2020 11:36:19 +0000 (14:36 +0300)
committerIlya Bizyaev <bizyaev@zoho.com>
Wed, 12 Feb 2020 15:58:47 +0000 (18:58 +0300)
commit402f7f3041d45d8c5969f3bc202fe9156891cd2e
tree74603188ecb0b108614446bd73dc635762eefb3d
parent2c29c7ed28f7389ec79907760311fce3a26622c2
Swap "Open in New Window" and "Open in New Tab" in context menus

Summary:
Since Dolphin's default behaviour became opening in new tabs rather than in new windows
(a very positive change, in my opinion), I always find myself accidentally opening folders
in new windows from the context menu. This is because in most browsers with tab workflow
(e.g. Firefox, Chromium, Falkon) the top context menu action is opening in a new tab, and
my muscle memory plays a bad trick :) I'm aware of middle-clicking, but I find it
inconvenient with my touchpad.

I suspect I'm probably not alone in that, so I think it makes sense to swap these actions
in Dolphin to match widely used apps.

Test Plan:
Open Dolphin, right click folders in the main view and in the Places panel, ensure the
actions are indeed swapped

Reviewers: #vdg, ndavis, #dolphin, ngraham

Reviewed By: #vdg, ndavis, #dolphin, ngraham

Subscribers: ngraham, ndavis, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D27318
src/dolphincontextmenu.cpp
src/panels/places/placespanel.cpp