Add unique-window mode which opens uris in a new tab instead of a window
AbandonedPublic

Authored by amhndu on Mar 19 2019, 1:41 PM.

Details

Reviewers
None
Group Reviewers
Dolphin
Summary

Newly launched dolphin windows will close and the given files will be
opened in the previously opened window.

This can be overriden with the --new-window command line option and gives
flexibility to the user so that they can still create another window as they could
previously.

FEATURE: 183429

Test Plan

Go to Settings -> Startup and toggle the mode on.
Restart Dolphin and then try opening another instance.

Diff Detail

Repository
R318 Dolphin
Branch
unique-window
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9847
Build 9865: arc lint + arc unit
amhndu created this revision.Mar 19 2019, 1:41 PM
Restricted Application added a project: Dolphin. · View Herald TranscriptMar 19 2019, 1:41 PM
Restricted Application added a subscriber: kfm-devel. · View Herald Transcript
amhndu requested review of this revision.Mar 19 2019, 1:41 PM

This is now the third patch we've received that implements this feature, the others being D16648 and D19150. I guess people really want it!

@elvisangelaccio, as maintainer, could you sort out our path forward here?

I was not aware there were patches that already did this.
I only found https://git.reviewboard.kde.org/r/129043/ on the bug report which was abandoned, otherwise I would not have created another one!

No worries, the interest is appreciated anyway! I've updated the bug report so hopefully even more people don't fall into this situation. :) @elvisangelaccio will sort the patches out, and in the meantime, Dolphin has many other interesting projects that still need patches. Check out some of these: https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&keywords=usability%2C%20junior-jobs%2C%20&keywords_type=allwords&list_id=1599062&product=dolphin&product=frameworks-baloo&product=frameworks-kio&query_format=advanced

Sorry for the delay. Let's focus on D16648 please, which has already started its review process.

amhndu abandoned this revision.Apr 12 2019, 4:57 AM

I understand