<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/105796/">http://git.reviewboard.kde.org/r/105796/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 30th, 2012, 9:42 p.m., <b>Francesco Cecconi</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 slider is used to suggest a quick range of font sizes (9/22), but this is not the best way.
In dolphin for example, one combo with four options "Small, Medium, Large, Huge" is used for this scope.
The second way is better in my opinion.</pre>
</blockquote>
<p>On July 30th, 2012, 10:32 p.m., <b>Francesco Cecconi</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;">remove completely the "slider" is another solution. :)</pre>
</blockquote>
<p>On July 31st, 2012, 2:05 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;">Well I think dolphin's slider (icon size) is different from Konsole's. Also I don't imagine that people change konsole's text size by a huge amount. I'm always annoying w/ Konsole's slider and it often comes up w/ a fraction when it should just be integers. And you can also use the 'select font' dialog to change the size for huge changes.</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;">> I'm always annoying w/ Konsole's slider and it often comes up w/ a fraction when it should just be integers.
Yes, this thing is annoying.</pre>
<br />
<p>- Francesco</p>
<br />
<p>On July 30th, 2012, 2:50 p.m., Kurt Hindenburg 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 Kurt Hindenburg.</div>
<p style="color: grey;"><i>Updated July 30, 2012, 2:50 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;">I'm not sure why a slider was used for the font size. And why fractions were allowed as Konsole has to have integer sizes.
If we really want a slider, kintnuminput has the option to add it as well.
I also removed the "size :" from the preview label as I was looking at bko 174998.
</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/EditProfileDialog.ui <span style="color: grey">(be3e672)</span></li>
<li>src/EditProfileDialog.h <span style="color: grey">(4715ec5)</span></li>
<li>src/EditProfileDialog.cpp <span style="color: grey">(68d5729)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105796/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>