Add a line to remember file paths when uploading title clips

Authored by akhilkgangadharan on Jan 5 2019, 6:45 PM.



BUG: T9905
Update variable m_projectTitlePath with user selected directory when the user tries to load title clips from outside the default directory.

Diff Detail

R158 Kdenlive
No Linters Available
No Unit Test Coverage
Build Status
Buildable 6681
Build 6699: arc lint + arc unit
akhilkgangadharan requested review of this revision.Jan 5 2019, 6:45 PM
akhilkgangadharan created this revision.
emohr added a comment.Jan 5 2019, 7:05 PM

Hi Akhilkgangadharan. I'm not a coder. Please add Jean-Baptiste Mardelle as reviewer. Thanks.

Thanks and sorry for the trouble emohr.

akhilkgangadharan edited reviewers, added: mardelle; removed: emohr.Jan 5 2019, 7:10 PM

Thanks for this contribution. To convert an url to a string, it is recommended to use toLocalFile() instead of toString() .
Also, the m_projectTitlePath variable is local to the titleWidget, so it is lost as soon as you close the title widget. To work around that, I used a KDE class that was designed to store paths for file dialogs: KRecentDirs::dir(QStringLiteral(":KdenliveProjectsTitles"));

I just pushed a complete fix for this issue:

akhilkgangadharan abandoned this revision.Jan 6 2019, 3:56 PM

Thanks for reviewing!
I didn't think of storing the path as I thought it wouldn't be needed but now thinking about it, it is required :)

Thanks for the suggestions, I'll abandon this revision.