<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/112078/">http://git.reviewboard.kde.org/r/112078/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On August 14th, 2013, 12:49 p.m. UTC, <b>Aleix Pol Gonzalez</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 don't think this is an obvious work-flow. I understand that it could useful but I don't think that removing the check is going to improve our life at all.
The user needs some feedback at least, that whenever he saves the document he'll be actually creating it.</pre>
</blockquote>
<p>On August 14th, 2013, 1:09 p.m. UTC, <b>Jørgen Kvalsvik</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;">Do you mean feedback as in it won't actually be created until he saves? I suppose that would be ok, especially if we can add a "Do not notify me again"-option.</pre>
</blockquote>
<p>On August 15th, 2013, 7:45 a.m. UTC, <b>Jørgen Kvalsvik</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 considered this for a while. The not-creating-before-it-is-saved actually applies to when you create a new file/buffer with Ctrl+N as well, so there is no reason for it to be different if you open with Ctrl+O. The fact that the semantic here is "open" also implies that it won't actually be written to disk before you flush it.</pre>
</blockquote>
<p>On August 15th, 2013, 5:21 p.m. UTC, <b>Jørgen Kvalsvik</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;">Oh, and Kate already behaves like this.</pre>
</blockquote>
<p>On August 16th, 2013, 9:57 a.m. UTC, <b>Aleix Pol Gonzalez</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;">No, with Ctrl+N you're being explicit.
Also it doesn't have to be a message box, you can use Kate's notifying system.
The fact that Kate behaves like this at the moment doesn't make it any better ;). In fact, it's a fix that probably should go in Kate. I think this review can wait until then, because it's quite simple and it will bring an overall better look&feel.</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;">You're being explicit, yes, but the file itself won't be written to the file system before you flush the buffer. Like every other editor I can think of.
A notification is OK, but in that case Kate should get it too, in the name of consistency. Right now KDevelop is the odd one out NOT opening a new buffer with the open-file-dialogue.</pre>
<br />
<p>- Jørgen</p>
<br />
<p>On August 14th, 2013, 10:04 a.m. UTC, Jørgen Kvalsvik 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 KDevelop.</div>
<div>By Jørgen Kvalsvik.</div>
<p style="color: grey;"><i>Updated Aug. 14, 2013, 10:04 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;">Removes the check if a file exists (KIO::NetAccess::exists()) so that the following routines will create the file instead of just attempting to open it.</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 both opening and creating files through the open file dialogue. Confirmed that creating files with File->New works as well.</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=322678">322678</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>shell/documentcontroller.cpp <span style="color: grey">(4fd2819)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/112078/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>