<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/111091/">http://git.reviewboard.kde.org/r/111091/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 20th, 2013, 12:49 p.m. UTC, <b>Daniele E. Domenichelli</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;">Sorry I still have a couple of issues:
1) Instead of reading and saving the lastDownloadDirectory entry you can use the kfiledialog:///XXXXXXX syntax for file dialog (see ktp-contact-list/contact-list-widget.cpp:371). You can also use the static methods for KFileDialog, so that you don't need to add a pointer to the class.
2) I don't like at all that if you enable the "Always Ask" flag you get first a dialog to choose the directory and then, if the file exists, a second dialog to rename the file. Imho, if you enable the flag you should get just one file chooser dialog, with a warning if you are overwriting an existing file, and you should skip the checkFileExists() call and go straight to checkPartFile();</pre>
</blockquote>
<p>On June 21st, 2013, 9:17 a.m. UTC, <b>Dan Vrátil</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'm using KDirSelectDialog, which is slightly different from KFileDialog (it's not a subclass)
ad 1) didn't know about that and it works with KDirSelectDialog, so it's fine
ad 2) this can't be solved when using KDirSelectDialog. And using KFileDialog (from user's POV) is slightly less comfortable than KDirSelectDialog when it comes to just selecting a directory. Also this option is about picking a directory, not a filename. Otherwise we would have to show the filename dialog every time.</pre>
</blockquote>
<p>On June 21st, 2013, 12:01 p.m. UTC, <b>Thomas Pfeiffer</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;">Why do you use KDirSelectDialog?
Using a normal KFileDialog makes perfect sense in this case, because users may indeed want to save the file under a different name, just like in a browser. From a user's point of view there is pretty much no difference between saving a file in a browser and saving a file received via KTp. It's not about choosing a folder, it's about choosing where and with which file name to save it.
Therefore the suggestion from the usability perspective is to emulate the browser behavior (you may take Rekonq as an example) as closely as possible. It works for browers and it would work for KTp just as well.
And if it's currently not technically possible to save a file received via KTp under a different name, than KTp is broken and needs to be fixed.</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 agree, it should not just let you choose the folder, but the whole path + file name, using by default the one received from the file-transfer (you can get it with channel->fileName())
I'm not sure if the kfiledialog:///XXXXXXX can handle this though, perhaps if you add the name after (something like kfiledialog:///XXXXXXX/filename.ext) it might save just the directory, you should try ;)</pre>
<br />
<p>- Daniele E.</p>
<br />
<p>On June 20th, 2013, 9:34 a.m. UTC, Dan Vrátil wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By Dan Vrátil.</div>
<p style="color: grey;"><i>Updated June 20, 2013, 9:34 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;">Implements support for "Always Ask" for filetransfer destination directory option.
The hack with __k__onDownloadDirectoryCancelled() is necessary because KDirSelectDialog::result() always returns 0 and buttonClicked() signal is emitted even when "New Folder" button is clicked and handling that would make the code more complicated.
</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=295937">295937</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/CMakeLists.txt <span style="color: grey">(ebd892c)</span></li>
<li>src/filetransfer-handler.cpp <span style="color: grey">(74c701e)</span></li>
<li>src/handle-incoming-file-transfer-channel-job.h <span style="color: grey">(d5e53c4)</span></li>
<li>src/handle-incoming-file-transfer-channel-job.cpp <span style="color: grey">(7a15237)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/111091/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>