<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/106252/">http://git.reviewboard.kde.org/r/106252/</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 3rd, 2012, 9:48 p.m., <b>Jarosław Staniek</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;">A number of suggestions to fight clutter a bit:
- Should 'East asian characters' be visible by default? If there are o asian characters maybe you could hide the label.
- Flesch reading ease -> Reading ease
(I noted this before and it was marked as fixed but is this fixed? I see no comments on that, and how this scientific term would be translated e.g. to German or Polish and still generally usable.)
- Characters (excluding spaces) -> Characters
(it's obvious for non-techies that characters are not spaces, and the the other value explicitly includes spaces)</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 guess 'East asian characters' is currently not visible by default (at least it wasn't in 2.5). If it is in master, that should be reversed of course.
- I suggested to Shreya to use "Readability" as label with "Flesch Reading Ease" as a Tooltip. Would that work?
- No, it is _not_ obvious to non-techies that characters does not include spaces. I just asked a "non-techie" and she thought that it _does_ include spaces, because she knows character counts mainly from character limits in forms or text messages, which usually do include spaces. I looked at Libre Office, and they have "Characters" and "Characters exlucing spaces", so they include them by default as well. Google docs has "Characters (with spaces)" and "Characters (no spaces)", so they obviously were undecided. I'd suggest we should go with the Google Docs labeling as it is unambiguous but not as long as the current labels.</pre>
<br />
<p>- Thomas</p>
<br />
<p>On August 31st, 2012, 12:19 p.m., Shreya Pandit 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 Calligra, Thomas Pfeiffer and Inge Wallin.</div>
<div>By Shreya Pandit.</div>
<p style="color: grey;"><i>Updated Aug. 31, 2012, 12:19 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;">This allows the Statistics Docker to update its layout horizontally, when placed in bottom area. </pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Checked for calligra words and calligra author</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>words/part/dockers/KWStatistics.h <span style="color: grey">(c480502)</span></li>
<li>words/part/dockers/KWStatistics.cpp <span style="color: grey">(a006d97)</span></li>
<li>words/part/dockers/KWStatisticsDocker.h <span style="color: grey">(fab6c7f)</span></li>
<li>words/part/dockers/KWStatisticsDocker.cpp <span style="color: grey">(0bc8130)</span></li>
<li>words/part/dockers/KWStatisticsDocker.ui <span style="color: grey">(63eaee5)</span></li>
<li>words/part/dockers/StatisticsPreferencesPopup.h <span style="color: grey">(b481806)</span></li>
<li>words/part/dockers/StatisticsPreferencesPopup.cpp <span style="color: grey">(c741622)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106252/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/106252/s/713/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/08/31/show_review_400x100.jpeg" style="border: 1px black solid;" alt="The layout" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>