Summary: | Renaming files/directories in folder view throws an error message | ||
---|---|---|---|
Product: | [Plasma] plasma4 | Reporter: | FiNeX <finex> |
Component: | widget-folderview | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | jpr, sander.ernes |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: |
remove KFileItemDelegate::EventFilter from editor before (possibly) grabbing focus.
modified patch for current trunk. |
Description
FiNeX
2008-10-01 13:39:35 UTC
Created attachment 27668 [details]
remove KFileItemDelegate::EventFilter from editor before (possibly) grabbing focus.
KFileItemDelegate::eventFilter emits commitData() and closeEditor() both on losing focus and pressing "Enter". Because focus remains on editor after "Enter" is pressed and in that case FolderView::closeEditor grabs it, FolderView is commiting (renaming) twice. Using the patch suggested by Sander, renaming files doesn't throw the error message. The patch is ok to me. I've updated it because the code of folderview has a bit changed. Created attachment 28314 [details]
modified patch for current trunk.
SVN commit 894444 by aseigo: patch by Sander Ernes to fix an errant warning dialog BUG:171959 M +1 -1 iconview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=894444 |