<div class="gmail_quote">2010/8/26 Nicolás Alvarez <span dir="ltr"><<a href="mailto:nicolas.alvarez@gmail.com">nicolas.alvarez@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
When quitting KDevelop with modified files open, KDevelop shows a<br>
dialog asking if you want to save the files first. The button selected<br>
by default is "Save None", causing data loss if the user presses Space<br>
or Enter when the dialog appears.<br>
<br>
The code calls setDefaultButton(Ok), but this doesn't work as<br>
expected. It means if the user presses Enter on the dialog, the button<br>
activated will be "OK". However, if a button is *focused*, the Enter<br>
key would activate the button that has focus instead. In this case,<br>
the "default button" only applies if focus is on the document list<br>
(the only control in the dialog apart from the buttons). Since Save<br>
None is the button with focus when the dialog opens, that's what Enter<br>
activates if user doesn't change focus.<br>
<br>
To do the right thing, we should (also) call setButtonFocus when the<br>
dialog is created.<br>
<br>
However, Kate has "Cancel" as both default button and<br>
initially-focused button, not "OK". It may be a good idea to do the<br>
same, at least for consistency.<br>
<br>
I also thought of leaving Ok as default button (to activate if Enter<br>
pressed with list focused), and making Cancel the initial-focus<br>
button, but I don't know if this difference may end up confusing for<br>
users.<br>
<br>
Thoughts?<br>
<br>
--<br>
Nicolas<br>
<font color="#888888"><br>
--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</font></blockquote></div><br><div>Hi Nicolás!</div><div>I just tried it and by default I get the "Save Modified" button selected (when i press enter/space it gets selected), maybe it's a bug in different KDialog revisions?</div>
<div><br></div><div>On what Kate does, I'm not sure what do they think but IMO save is a good default. I don't see a reason to have something else, in the end if you didn't want to save you can always revert, and if you didn't commit... then you should have :P. (in kdevelop's case most time files will be versioned, probably not in Kate).</div>
<div><br></div><div>Aleix</div><div><br></div>