[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    [PATCH] KFileWidget locationEdit issue
From:       "alessandro diaferia" <alediaferia () gmail ! com>
Date:       2008-06-16 17:42:00
Message-ID: 65627f3a0806161041m65a8d61di8a8e7d1645d85bab () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


 Hello,
I've just encountered a nasty bug in kfiledialog (open mode) when you have
some history. The steps to reproduce are:

1 - Start a KDE app which uses kfiledialog (e.g. KWrite) possibly with
recent *opened* files.
2 - Click on 'open'.
3 - Type the name of a folder you can see in the dialog's view.
3 - Press Enter.

 Result: the first file in the history opens.

Expected behavior: the dialog should 'cd' into the folder typed, and it
shouldn't open any file (especially because it is not even telling you what
the file that is going to be opened is ;-) ).
Currently, if you press the 'Open' button the behiavour is correct.

The patch should fix. Can someone please review it and tell me if it's ok to
commit?

Regards
-- 
Alessandro

[Attachment #5 (text/html)]

Hello,<br>
I&#39;ve just encountered a nasty bug in kfiledialog (open mode) when you \
have<br>some history. The steps to reproduce are:<br> <br>1 - Start a KDE app which \
uses kfiledialog (e.g. KWrite) possibly with<br>recent *opened* files.<br>2 - Click \
on &#39;open&#39;.<br>3 - Type the name of a folder you can see in the dialog&#39;s \
view.<br><div class="Ih2E3d">

3 - Press Enter.<br>
<br>
</div>&nbsp;Result: the first file in the history opens.<br>
<br>Expected behavior: the dialog should &#39;cd&#39; into the folder typed, and \
it<br>shouldn&#39;t open any file (especially because it is not even telling you \
what<br>the file that is going to be opened is ;-) ).<br> Currently, if you press the \
&#39;Open&#39; button the behiavour is correct.<br> <br>The patch should fix. Can \
someone please review it and tell me if it&#39;s ok to commit?<br> <br>Regards<br>-- \
<br>Alessandro


["kfilewidget_patch.diff" (application/octet-stream)]

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic