<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/103809/">http://git.reviewboard.kde.org/r/103809/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I agree with the patch and that a 'Clone Tab' is likely the best way of handling the confusion on what New Tab does.</pre>
<br />
<p>- Kurt</p>
<br />
<p>On January 28th, 2012, 2:54 p.m., Jekyll Wu wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Konsole.</div>
<div>By Jekyll Wu.</div>
<p style="color: grey;"><i>Updated Jan. 28, 2012, 2:54 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Currently, "New Tab" action will create new tab using the _defaultProfile of MainWindow. In normal usage, that does not cause differece than using _defaultProfile of SessionManager. However, when "--profile" option is used, _defaultProfile of MainWindow refers to the profile specifed on the command line. So in that case, "New Tab" is more like "Clone Tab".
I think "New Tab" should always open new tab using the default profile, no matter what command line options are given. That is the most simple and intuitive behavior.
Another argument is consistency. Since bug 184788 has been fixed, "New Tab" now behaves like "New Tab" in the case of "-e" option, but more like "Clone Tab" in the case of "--profile" option. I think that is only confusing.
The patch is simple. However, it is the first step towards getting rid of _defaultProfile from MainWindow. So It is better to discuss the idea first here.
My general thought is making "New Tab" action simple and adding "Clone Tab" action as its complement, as posted in bug 292518 and bug 254976
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/MainWindow.cpp <span style="color: grey">(745b51f)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103809/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>