[Kde-pim] Review Request 112257: Add kontact dock status to session restore

Dan Vrátil dvratil at redhat.com
Fri Aug 30 14:27:47 BST 2013


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


Without really checking the code, this might not work as expected, because Kontact itself does not have any tray icon. The tray icons are provided by KMail or Akregator (or others), so even if you restore this value, KMail can still decide to show the window anyway (because it does not know about this).

- Dan Vrátil


On Aug. 25, 2013, 2:36 a.m., Lindsay Mathieson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112257/
> -----------------------------------------------------------
> 
> (Updated Aug. 25, 2013, 2:36 a.m.)
> 
> 
> Review request for KDEPIM and Lindsay Mathieson.
> 
> 
> Description
> -------
> 
> Kontacts dock status in the system tray is not preserved on session
> restore. This patch saves the docs status and checks it on session
> restore.
> 
> 
> Diffs
> -----
> 
>   kontact/src/main.cpp 54bb1b7 
>   kontact/src/mainwindow.cpp 6e8f09d 
> 
> Diff: http://git.reviewboard.kde.org/r/112257/diff/
> 
> 
> Testing
> -------
> 
> Logging out/in with kontact displayed and hidden in the system tray. Both states were correctly restored.
> Additionaly, checked that std restore from restarting kontact was working correctly.
> 
> 
> Thanks,
> 
> Lindsay Mathieson
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


More information about the kde-pim mailing list