<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://svn.reviewboard.kde.org/r/6580/">http://svn.reviewboard.kde.org/r/6580/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/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 kdeutils.</div>
<div>By peter velthuis.</div>
<p style="color: grey;"><i>Updated March 4, 2011, 4:07 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description (updated)</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;">-double click/enter: opens with default viewer
-'preview' button: opens the viewer like it does now.
-ctrl+w: 'open with' dialog shows to choose the application to open with. (it's
a KAction, but not added to a context menu or anything (yet)).
other:
QFile::remove calls removed and centralized in the destructor by deleting KTempDirs.
It creates a folder per opened file, same behaviour as winrar. Why?
The problem is that you dont know when an external program closes (started through KRun) so we cant remove the file directly after the external program closes. So with one directory this scenario will happen: open file, (temp dir created and file extracted therre), user closes file, opens again .. and the message "File already exists" appears. Probably this is also the first reason why the current preview dialog was created=> we know when the user closes it.</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="https://bugs.kde.org/show_bug.cgi?id=179066">179066</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>/trunk/KDE/kdeutils/ark/part/arkviewer.cpp <span style="color: grey">(1222951)</span></li>
<li>/trunk/KDE/kdeutils/ark/part/part.h <span style="color: grey">(1222951)</span></li>
<li>/trunk/KDE/kdeutils/ark/part/part.cpp <span style="color: grey">(1222951)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/6580/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>