Review Request: Plasmate:make plasmate remember the last checked qdockwidgets

Sebastian Kügler sebas at kde.org
Thu Jun 16 12:53:12 CEST 2011


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


Thanks for looking into this, first start appearance is very important. Unfortunately, I'm not too thrilled by it, as it influences the default view which should be rather clean. Remembering the state of dockwidgets should be completely transparant to the users, if we have to enforce it by showing them all, that's counter-intuitive. There has to be a better way to achieve this.

Maybe ask on kde-devel at kde.org if anybody knows a better solution?

- Sebastian


On June 15, 2011, 9:33 a.m., Giorgos Tsiapaliwkas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101582/
> -----------------------------------------------------------
> 
> (Updated June 15, 2011, 9:33 a.m.)
> 
> 
> Review request for Plasma and Aaron J. Seigo.
> 
> 
> Summary
> -------
> 
> hello,
> 
> since now plasmate wasn't able to remember the last used qdockwidgets,except from 1 or 2.
> With this patch,it (plasmate) is able to remember the last state of the qdockwidgets and also the size of the mainwindow (maximized or not).
> The drawback is that when the plasmate will start for the first time it will be a bit too populated since all the qdockwidgets will be checked.
> 
> 
> Diffs
> -----
> 
>   mainwindow.cpp 8326a29 
> 
> Diff: http://git.reviewboard.kde.org/r/101582/diff
> 
> 
> Testing
> -------
> 
> compiles fine without issues and regressions.
> 
> 
> Thanks,
> 
> Giorgos
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110616/309a4202/attachment-0001.htm 


More information about the Plasma-devel mailing list