D5183: Reopen accidentally closed tab

[Alexander Basov]

coopht marked 7 inline comments as done.
coopht added inline comments.

INLINE COMMENTS

> mwolff wrote in container.cpp:367
> this could potentially end up storing tons of items... I wonder - could we instead rely on the "recent files" mechanism and only support reopening the last N items through that?

I did not find proper way to use "recent files", because this list is application wide and we need to track separate queue of closed files for each container. 
And also files are added to the "recent files" list on file open event, which is not very usable in our case. 
I can limit the queue of closed files by 100 items. Is it enough?

REVISION DETAIL
  https://phabricator.kde.org/D5183

To: coopht, #kdevelop, mwolff
Cc: mwolff, brauch, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170326/5bce99b7/attachment.html>