<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/102688/">http://git.reviewboard.kde.org/r/102688/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 24th, 2011, 2:20 p.m., <b>Kurt Hindenburg</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The 'save to current profile' was put in there as there were complaints from users. This was back when the History was in the main menu, not just context menu.
I basically agree w/ your argument but still think there will be confusion. Putting some text say this change is temporary is likely a bit much.
Should the 'Default' put the settings back to the profile settings?</pre>
</blockquote>
<p>On September 26th, 2011, 8:01 a.m., <b>Jekyll Wu</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Aboot the 'save to current profile': I agree putting some text label is useful and necessary and will add it.
About the 'Default' button: the main reason I want to remove it is what it means is confusing to users, both in concept and implementation.
In concept: What is that default value? the value in the profile ? or the value in use before I open this dialog? or some value hardcode in the code, i.e., the default of default?
In implementation: the current code use hardcoded 'fixed history' and '1000 lines' as the default(the 3rd possibility metiond above). I think that is the least intuitive choice.
I guess a 'Reset' button is a better choice: it just resets those checkboxes to the state when this dialog is opened.
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Well, I am really not good at UI stuff. I failed to find a way to make clear the temporary intention while keeping the dialog compact and elegant.
If this change should go into the mainline, I think the corresponding context menu entry also need some change :
1). It says "Configure Scrollback...", where "Configure" generally implies persistent change
2). It uses the 'configure' icon, which also generally implies persistent change
</pre>
<br />
<p>- Jekyll</p>
<br />
<p>On September 24th, 2011, 9:45 a.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 Sept. 24, 2011, 9:45 a.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;">The main changes :
- the change is not saved into profile and only influences current tab
- remove the 'save into profile' checkbox
- remove the 'Default' button
My main argument is current design is (potentially) confusing and not consistent with other actions in the context menu. Most actions(Encoding, Clear Scrollback, Change profile) in the context menu are per tab, which means they only influce current tab and does not update the profile file. That is why the menu is called as context menu.
In general, I think the 'Edit Profile' dialog should be the only place where modification can be saved persistently. That would simplify both the concept and implementation.
</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=227114">227114</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/HistorySizeDialog.h <span style="color: grey">(e4ee4b2)</span></li>
<li>src/HistorySizeDialog.cpp <span style="color: grey">(79ccaa9)</span></li>
<li>src/SessionController.h <span style="color: grey">(796b579)</span></li>
<li>src/SessionController.cpp <span style="color: grey">(d109bb2)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102688/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/102688/s/270/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/24/empty-name-waring-right-below-profile-name2_400x100.png" style="border: 1px black solid;" alt="HistorySizeDialog" /></a>
<a href="http://git.reviewboard.kde.org/r/102688/s/272/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/25/utilities1_400x100.png" style="border: 1px black solid;" alt="using combobox instead of checkbox" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>