Review Request 118968: Use DolphinTabPage saveState/restoreState to remember and re-open closed tabs.

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Fri Jun 27 02:20:06 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118968/
-----------------------------------------------------------

(Updated June 27, 2014, 3:20 a.m.)


Review request for Dolphin.


Changes
-------

Added missing branch


Repository: kde-baseapps


Description
-------

Use DolphinTabPage saveState/restoreState to remember and re-open closed tabs.

Remembers: primary url, secondary url, splitter state, url navigator editable state and active view.

Advantages:
* Reuse code/save-restore-logic
* Makes testing of the tabs save-restore-logic easier ("application-level session management" ;)
* Tabs are "properly" restored (in future we can/should also save the DolphinView states)


Diffs
-----

  dolphin/src/dolphinmainwindow.h acf60a4 
  dolphin/src/dolphinmainwindow.cpp c60951d 
  dolphin/src/dolphinrecenttabsmenu.h 34d4153 
  dolphin/src/dolphinrecenttabsmenu.cpp a39f994 

Diff: https://git.reviewboard.kde.org/r/118968/diff/


Testing
-------

Restoring active view doesn't work (will fixed it), but everything else works.

Closing a tab with enabled split view and modified splitter (not centered anymore) and
reopen it in recent tabs menu restores the tab, with the two views and right splitter position.


Thanks,

Emmanuel Pescosta

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20140627/ab207a62/attachment.htm>


More information about the kfm-devel mailing list