KDirSelectDialog change icon size feature...

Shaun Reich predator106 at gmail.com
Sat Mar 14 19:02:40 GMT 2009

Hello, this is a patch that I need to get some feedback on, it is for
kdelibs/kfile/kdirselectdialog.cpp & .h.

I have a question, also...

Why, in the open file dialog, do the zoom in and zoom out actions on
the toolbar, they have their text set to "Zoom In" and "Zoom Out", but
it is not shown.

So it looks like the intention was to have the text there, but I do
not know why it is not. I am referring most directly to
kfilewidget.cpp. So I am not sure if it does not use that (I thought
it did), or if it is a bug, or if it is a mistake that the text should
be nil.

I added this feature because you can set the icon size in the open
file dialog and I want it to be a feature in this folder dialog.
Mostly, due in part because I do not like the default icon size, and
like to be able to change it to whatever I want.

Ignore the qDebug(), it won't be a part of it...

What do you guys think? It may be kind of sloppy still, not sure...

Riverenter Vestri,
Shaun Reich
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdirselectdialog.cpp.diff
Type: text/x-patch
Size: 4021 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090314/946baf53/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdirselectdialog.h.diff
Type: text/x-patch
Size: 517 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090314/946baf53/attachment-0001.bin>

More information about the kde-core-devel mailing list