Bug 364719

Summary: git-master-2016-06-25: Render window does not remember last rendered directory
Product: [Applications] kdenlive Reporter: Evert Vorster <evorster>
Component: User Interface & MiscellaneousAssignee: Jean-Baptiste Mardelle <jb>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Evert Vorster 2016-06-24 19:25:13 UTC
It's only a small niggle, until you are in a huge directory structure. 
When hitting the "Browse" button in the render widget, it always starts at the same directory. 
It would be much nicer if the browse widget started at the directory already typed in, if it exists. 

Reproducible: Always

Steps to Reproduce:
1. Render one file in a very deep directory structure. 
2. Click on "Browse" button in render widget to navigate to just one directory up or down. 

Actual Results:  
The browse always starts at the same directory

Expected Results:  
The Browser should start in the directory you rendered in last, ie the one that is already in the widget.
Comment 1 Jean-Baptiste Mardelle 2016-06-29 10:12:04 UTC
Seems to be a bug in KDE's KUrlRequester.
Following patch fixes it for me, I am going to create a ticket against KIO.

diff --git a/src/widgets/kurlrequester.cpp b/src/widgets/kurlrequester.cpp
index 2d87d66..fc87c8f 100644
--- a/src/widgets/kurlrequester.cpp
+++ b/src/widgets/kurlrequester.cpp
@@ -432,6 +432,7 @@ void KUrlRequester::KUrlRequesterPrivate::_k_slotOpenDialog()
             QUrl u(url());
             // If we won't be able to list it (e.g. http), then don't try :)
             if (KProtocolManager::supportsListing(u)) {
+                dlg->setDirectoryUrl(u.adjusted(QUrl::RemoveFilename));
                 dlg->selectUrl(u);
             }
         } else {
Comment 2 Evert Vorster 2016-06-29 11:52:16 UTC
Thanks for looking into this for me. 
I'll wait for the next update of KDE, as I am in no mood to start running KDE from git. In the meantime I will close this ticket. 

Thanks!
Comment 3 Jean-Baptiste Mardelle 2016-07-02 08:14:32 UTC
Git commit 580dfd4992ca7d1d472780e8148f9fbd04d73c92 by Jean-Baptiste Mardelle.
Committed on 02/07/2016 at 08:10.
Pushed by mardelle into branch 'master'.

Fix KUrlRequester opening file dialog in wrong directory
REVIEW: 128315

M  +1    -0    src/widgets/kurlrequester.cpp

http://commits.kde.org/kio/580dfd4992ca7d1d472780e8148f9fbd04d73c92