Open a new tab instead of closing the app in dropdown mode

[f2404]

Fixes #154

[jleclanche]

That... doesn't seem right at all.

[f2404]

Why? I see it's implementing the request.

[pvanek]

It seems strange to me. But I'm not dropdown user...

[agaida]

@f2494 - could you resolve the conflicts please - @pvanek: looks strange, but is imho the right and expected behaviour.

[f2404]

@agaida Rebased.

[agaida]

tested this and see to many glitches - the idea is good, but the code not (yet)
https://bugs.siduction.org/attachments/download/272/vokoscreen-2016-10-05_20-06-27.mkv

Sorry, was to fast with my approval - and that lead to the next question - how to undo/remove an approval?

[f2404]

@agaida Please check the latest commit.

[agaida]

@f2404: Works for me

@palinek @pmattern @paulolieuthier - could you test too?

[pmattern]

Closing a second (regular) window which had been opened from a QTerminal "window" in drop-mode before terminates the QTerminal instance altogether. Don't know whether this is expected behavior.

That aside f2404/qterminal@aa01554e seems to behave as asked for in #154. But I'm probably not the right person to judge that whole thing because I still think the behavior proposed here is just plain wrong.

[pmattern]

Meanwhile I do think this new feature is a good thing as well.

@f2404
(Assuming you are not too annoyed by this PR celebrating its first birthday some days ago:)
What I wrote above (first paragraph) wasn't accurate. When another "regular" instance is launched by invoking qterminal & in a drop-down terminal and this additional instance is terminated, the instance in drop-down mode is not terminated as well. Rather, the key to make the drop-down instance visible seems to get disabled. Isn't this worth having another look at?
Any chance you can turn the new feature into an option? Could very well be enabled by default. As for the location within the settings I'd suggest an additional new topic at Behaviour (left pane) - Behaviour (section in right pane).
Sure enough the commits have to be rebased by now as well.

all in all not the right approach