Review Request: Add Window Manager Session Id to Restore Command

[David Narváez]

> On Oct. 3, 2011, 12:16 p.m., Aleix Pol Gonzalez wrote:
> > I'm not sure how sessions work, but don't you think that the x11 session should save the session to restore state?

It does, but by overriding the restore command (in our case, with the -s <session-id> command) you effectively take away the x11 session information to restore. The patch actually builds the original command (which is -session <x11-session-id>) and adds the KDevelop session information.


- David


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


On Oct. 3, 2011, 11:50 a.m., David Narváez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102759/
> -----------------------------------------------------------
> 
> (Updated Oct. 3, 2011, 11:50 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> Without this information, KDevelop won't be restored in the correct Desktop/Activity.
> 
> 
> Diffs
> -----
> 
>   app/main.cpp 89d67a4 
> 
> Diff: http://git.reviewboard.kde.org/r/102759/diff/diff
> 
> 
> Testing
> -------
> 
> 1) Open KDevelop in Activity A, Desktop a
> 2) Change to Activity B
> 3) Logout and Login
> 4) You should be in Activity B and KDevelop should have been restored at Desktop a of Activity A
> 
> 
> Thanks,
> 
> David Narváez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111003/0260dbf4/attachment.html>