Review Request: Allow selecting multiple toolviews to be added in the new toolview dialog.

[Milian Wolff]

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

Ship it!


sounds good. also the idea about adding the toolview on double click (please test whether that works)

thing is though that one can currently have the same view multiple times. we might think about changing that to at least change that to "one view *per area*". please start a thread on our -devel ml on this topic

- Milian Wolff


On May 1, 2012, 9:28 p.m., Ghislain Mary wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104818/
> -----------------------------------------------------------
> 
> (Updated May 1, 2012, 9:28 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> Allow selecting multiple toolviews to be added in the new toolview dialog.
> 
> As I understand it, the complaint from this bug report is that one has to open the new toolview dialog several times to add new toolviews. So I enabled the selection of multiple toolviews to be added in the dialog. I think it would not be a good user experience to add a toolview on a double click without closing the dialog since the user may not see that the toolview has indeed been added. What do you think?
> By the way, does it make sense to be able to add a toolview several times? Because if a toolview should be present only one time, maybe an user interface like the one to customize the toolbar (with two lists and arrows to pass elements from one list to the other) would be more relevant.
> 
> 
> This addresses bug 165969.
>     http://bugs.kde.org/show_bug.cgi?id=165969
> 
> 
> Diffs
> -----
> 
>   shell/uicontroller.cpp ad0e583 
> 
> Diff: http://git.reviewboard.kde.org/r/104818/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ghislain Mary
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120503/226828d2/attachment.html>