<div><div dir="auto">im Open to redo the patch but I don’t see the problems you guys see. The only thing this patch does is to quit early if you trigger a unique application *after* a multiple one has been spawned.</div></div><div dir="auto"><br></div><div dir="auto">See:</div><div dir="auto">Konsole (register as multiple)</div><div dir="auto">Konsole —new-tab (register as single)</div><div dir="auto"><br></div><div dir="auto">But instead of adding a tab to the old, multiple, instance - it opens a new window instead.</div><div dir="auto"><br></div><div dir="auto">I’ve tested with multiple single ones, and mixing singles with multiples So I don’t understand why you say it would break anything.</div><div dir="auto"><br></div><div dir="auto">Also, is there anything that I can do to fix this konsole bug I explained any other way?</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 3 Nov 2019 at 11:01 David Faure <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><table><tbody><tr><td>dfaure requested changes to this revision.<br>dfaure added a comment.<br>This revision now requires changes to proceed.
</td><td><a style="text-decoration:none;padding:4px 8px;margin:0 8px 8px;float:right;color:#464c5c;font-weight:bold;border-radius:3px;background-color:#f7f7f9;background-image:linear-gradient(to bottom,#fff,#f1f0f1);display:inline-block;border:1px solid rgba(71,87,120,.2)" href="https://phabricator.kde.org/D25084" target="_blank">View Revision</a></td></tr></tbody></table><br><div><div><p>Same reservations as davidedmundson.<br>
Furthermore this would break talking to dbus-activated unique apps, too, since they wouldn't register with the expected service name.<br>
-2 from me.</p></div></div><br><div><strong>REPOSITORY</strong><div><div>R271 KDBusAddons</div></div></div><br><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25084" target="_blank">https://phabricator.kde.org/D25084</a></div></div><br><div><strong>To: </strong>tcanabrava, adridg, dfaure<br><strong>Cc: </strong>dfaure, GB_2, davidedmundson, adridg, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns<br></div></blockquote></div></div>