D27318: Swap "Open in New Window" and "Open in New Tab" in context menus

[Ilya Bizyaev]

IlyaBizyaev created this revision.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
IlyaBizyaev requested review of this revision.

REVISION 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

REPOSITORY
  R318 Dolphin

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D27318

AFFECTED FILES
  src/dolphincontextmenu.cpp
  src/panels/places/placespanel.cpp

To: IlyaBizyaev
Cc: kfm-devel, pberestov, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200211/d4ecbe13/attachment.htm>