<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/106650/">http://git.reviewboard.kde.org/r/106650/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 3rd, 2012, 7:38 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;">I propose that we use the opportunity of removing bloat[tm].
I think something could be changed, but not this move ("Show Statusbar" is in Settings menu in the never-finished KDE HIG). As usual, let's ask yourself and your testers, how frequently do you use the option.
I have personally seen users and myself rather calling full-screen action, if anything, in apps to get some space for my editor's content. So how about a setting in application settings dialog, not a prominent View option?
For special users that feel ignored by that, there are always the shortcut keys editor and the toolbar editor.</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 do agree with this. A configurable full-screen action like Krita has will be more useful. Toggling the visibility of the statusbar (or menubar) really isn't useful. On the other hand, consistency with other KDE applications is important all by itself. </pre>
<br />
<p>- Boudewijn</p>
<br />
<p>On September 30th, 2012, 12:10 a.m., Inge Wallin 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, Marijn Kruisselbrink, Thorsten Zachmann, and C. Boemann.</div>
<div>By Inge Wallin.</div>
<p style="color: grey;"><i>Updated Sept. 30, 2012, 12:10 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;">This patch moves the action "Show Statusbar" to the View menu.
It's obvious that to show the statusbar or not is a View thing.
One thing to comment on is whether all applications should have this setting. Currently they don't.</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;">Tested all affected applications.</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>sheets/sheets.rc <span style="color: grey">(7eae858)</span></li>
<li>stage/part/stage.rc <span style="color: grey">(2dba463)</span></li>
<li>stage/stage.rc <span style="color: grey">(d0b912f)</span></li>
<li>words/part/author/author.rc <span style="color: grey">(14c5e91)</span></li>
<li>words/part/words.rc <span style="color: grey">(7ccbd34)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106650/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>