[patch] konqi multi column view #42587

[David Faure]

On Friday 03 December 2004 21:52, Martin Koller wrote:
> Hi all,
> 
> I've another patch for konqi addressing
> http://bugs.kde.org/show_bug.cgi?id=42587
> 
> I solve this by not setting a fixed grid when the text is to the right of the 
> icon. Then the widget already does a good job in calculating the variable(!) 
> column width.
> 
> In addition, the user can set the maximum width (by using the TextWidth I 
> added recently to konq_settings, but now using it not for the desktop).
> 
> I also included this to the File Manager/Appearance kcontrol Module.
> 
> Can I commit ?

I'm not sure which approach is best, but did you see the patch from
Michael Mellor on kde-devel? It also addresses #42587 but with a different
solution. From a quick reading it doesn't set a max width, so I guess it 
probably disables wordwrap altogether?

> P.S.: And I also set the icons to non moveable in file manager mode as this 
> does not make any sense, I think.
Hmm, this is arguable. All file managers allow moving them, don't they?

PS: missing each other's patches wouldn't happen if this was discussed
on kfm-devel, which is the right list for this :)

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).