Bug 160400 - open/save dialogue does not remember the size of the preview-part
Summary: open/save dialogue does not remember the size of the preview-part
Status: RESOLVED FIXED
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Rafael Fernández López
URL:
Keywords:
: 173281 175591 185764 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-04-05 13:30 UTC by S. Burmeister
Modified: 2009-03-02 01:30 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
screenshot (38.86 KB, image/png)
2008-04-05 13:30 UTC, S. Burmeister
Details
Proposed patch (721 bytes, patch)
2009-01-29 23:03 UTC, Dario Andres
Details

Note You need to log in before you can comment on or make changes to this bug.
Description S. Burmeister 2008-04-05 13:30:33 UTC
Version:            (using KDE 4.0.3)
OS:                Linux

If I resize the preview-bit of the open/save dialogue, because it uses too much space, it does not remember this size. As you can see in the screenshot this forces the user to either increase the size of the whole dialogue or deal with a too small file-list.
Comment 1 S. Burmeister 2008-04-05 13:30:59 UTC
Created attachment 24194 [details]
screenshot
Comment 2 premierSullivan 2008-06-26 06:46:14 UTC
confirmed here... both the preview pane and the places pane are like goldfish.  They grow to fill the availible space. Over time it seems like they get bigger until the file list is unusable.  I adjust it back, but the next time I open it its broken again.
Comment 3 Dario Andres 2008-08-01 00:56:51 UTC
Using KDE 4.1.00 (KDE 4.1.0) (KDEmod) in ArchLinux i686:
I can reproduce this behaviour. The preview pane size isn't saved or reloaded.
Comment 4 Dario Andres 2008-08-19 18:28:00 UTC
Seems to be fixed in :
Version 4.1.62 (KDE 4.1.62 (KDE 4.2 >= 20080814))
kdelibs4.2 svn rev.849324
kdebase4.2 svn rev.849337
ArchLinux 2.6.25-zen2-20080814 i686 

I don't know if it's fixed in the KDE4.1.1+ branch
Comment 5 Dario Andres 2008-08-19 18:39:30 UTC
No, it isn't fixed even in KDE4.2 svn . I've reproduce it recently :(
Comment 6 Seb 2008-10-27 10:13:28 UTC
*** Bug 173281 has been marked as a duplicate of this bug. ***
Comment 7 Dario Andres 2008-12-26 15:03:56 UTC
Can anyone still reproduce this bug with a recent KDE version? (4.1.3 / 4.2beta2 / 4.2svn) ?  Thanks :) 
(The file dialogs changed a lot in KDE4.2 :) )
Comment 8 Ronny Standtke 2008-12-27 14:03:02 UTC
Yes, I can still reproduce this bug with KDE 4.2 Beta2:
Start kate, click "Open", scale down the preview part, close the open dialog, open it several times again and watch how the preview part grows bigger and bigger every time until the file list is unusable again.
Comment 9 Dario Andres 2009-01-07 21:08:15 UTC
*** Bug 175591 has been marked as a duplicate of this bug. ***
Comment 10 Ronny Standtke 2009-01-15 12:24:47 UTC
This bug is still present in KDE-4.2rc1.
Comment 11 Ronny Standtke 2009-01-28 21:10:12 UTC
This absolutely annoying bug made it's way to the final KDE-4.2. Is this "The Answer"?
Comment 12 Dario Andres 2009-01-29 23:03:22 UTC
Created attachment 30725 [details]
Proposed patch

@Ronny: can you test this patch ?
Comment 13 Dario Andres 2009-01-29 23:04:51 UTC
That patch avoids the preview widget to grow in size with every opening of the file dialog.
Comment 14 Dario Andres 2009-01-29 23:22:38 UTC
@Peter: acording to the annotations, you wrote the original code. Can you check out my patch?

(Debugging this, I discovered that with the original line location , "restorePreviewWidth" was always false (because the size were the same). So the previewWidget didn't got resized to the properly width, and therefore it was increasing its size with each filedialog opening). With my patch, it compares the size after the splitter resize (based on the filedialog size), and resizes the previewWidget correctly )

Thanks
Comment 15 Dario Andres 2009-01-29 23:59:15 UTC
I forgot something: the patch must be applied against kdelibs/kfile/kdiroperator.cpp
Comment 16 Peter Penz 2009-01-30 07:25:20 UTC
Thanks Darío for the patch, it looks OK from my point of view but I want to leave the decision up to Rafael (he is maintaining the file dialog quite a while already and has a better overview about it). I've assigned this issue to Rafael...
Comment 17 Rafael Fernández López 2009-02-03 11:53:44 UTC
I'm sorry for the late reply, I am really busy with university at the moment.

The patch looks fine, please go ahead on committing it.
Comment 18 Dario Andres 2009-02-03 12:36:49 UTC
SVN commit 920600 by darioandres:

Properly resize the preview-widget on dialog resizeEvent
BUG: 160400



 M  +2 -2      kdiroperator.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=920600
Comment 19 Dario Andres 2009-02-03 12:38:54 UTC
SVN commit 920601 by darioandres:

Backport of SVN commit 920600 to 4.2branch
Properly resize the preview-widget on dialog resizeEvent
BUG: 160400



 M  +2 -2      kdiroperator.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=920601
Comment 20 Ronny Standtke 2009-02-08 16:33:36 UTC
> @Ronny: can you test this patch ?
Sorry for the late reply. No, I can not test the patch because, unfortunately, I do not have time to build from source, etc. I just tested releases so far. But thankfully it looks like I no not need to test the patch anymore. Thank you Darío, Peter and Rafael!
Comment 21 Dario Andres 2009-03-02 01:30:27 UTC
*** Bug 185764 has been marked as a duplicate of this bug. ***