Review Request: Change in KFileItemDelegate's renaming / editor behavior for directories

Christian Muehlhaeuser muesli at
Sat Jul 3 05:26:27 BST 2010

This is an automatically generated e-mail. To reply, visit:

Review request for kdelibs and Fredrik Höglund.


As discussed with FredrikH:

When renaming an item, KFileItemDelegate excludes the extension from the initial text selection.
While this makes perfect sense for files, directories don't typically have an extension, which indicates a filetype. They're directories after all.

This patch keeps the existing behavior for files, but always selects the entire text for directories.

Note, that I'm afraid I couldn't test the patch compiling nicely on 4.5 yet.
Since it's "just a two-liner (tm)", I'm confident it works fine as attached here and I'd hope someone on here could confirm it compiling / working for me, please?

Please don't commit the patch just yet. For consistency's sake I'm also fixing the behavior inside Dolphin's own RenameDialog right now. I'm going to commit
both patches at the same time, then.

Thanks in advance,


  trunk/KDE/kdelibs/kio/kio/kfileitemdelegate.cpp 1145462 





-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list