[rekonq] Review Request: Fixing "wrong" directory shown by KFileDialog

Andrea Diamantini adjam7 at gmail.com
Sun Dec 16 10:44:04 UTC 2012



> On Dec. 9, 2012, 10:27 a.m., Andrea Diamantini wrote:
> > Hi Thomas, and sorry for the late reply.
> > I noticed the bug, but I don't think your patch IS the solution, also if it really seems to. In fact, your patch blocks all recent dirs uses, forcing download path choice. See bug:184202, reported in the code && KFileDialog code about.

Change committed in commit 12b3217f9b26e should fix the problem and respect KRecentDirs KFileDialog feature.


- Andrea


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107589/#review23187
-----------------------------------------------------------


On Dec. 5, 2012, 3:08 a.m., Thomas Murach wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107589/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2012, 3:08 a.m.)
> 
> 
> Review request for rekonq.
> 
> 
> Description
> -------
> 
> When downloading an arbitrary file, the KFileDialog that's opened shows the root directory, not the user's home directory or even better the download path that's been configured by the user.
> This patch changes the default directory which the dialog shows to the Rekonfig download directory.
> 
> 
> Diffs
> -----
> 
>   src/downloadmanager.cpp 8070ed5 
> 
> Diff: http://git.reviewboard.kde.org/r/107589/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Thomas Murach
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/rekonq/attachments/20121216/bbaa05e2/attachment.html>


More information about the rekonq mailing list