[Konsole-devel] Review Request: Get back the "Manage Profiles..." entry in the context menu of konsolepart

[Commit Hook]

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


This review has been submitted with commit 1ffeb4bb6d4c676c749240fb7d50ee59e376be3f by Jekyll Wu to branch master.

- Commit Hook


On March 22, 2012, 7:23 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104361/
> -----------------------------------------------------------
> 
> (Updated March 22, 2012, 7:23 a.m.)
> 
> 
> Review request for Konsole, Yakuake and Eike Hein.
> 
> 
> Description
> -------
> 
> That entry was there in the early versions of KDE 4.x(like 4.1), but was gone since KDE 4.7 or even earlier. 
> 
> The rationale for getting it back :
> 
>     * "Switch Profile" only lists profiles set as "Favorite" or "Show in menu"
> 
>     * each konsolepart has its own sets of favorites, instead of reusing the sets from stand-alone Konsole
> 
>     * The only way to change favorite status is using the "Manage Profiles" dialog
> 
>     * konsolepart provide a slot "showManageProfilesDialog(QWidget* parent)" for show that dialog, but most hosting applications do not take advantage of it and provide an GUI item. Yakuake, Konversation and QuadKonsole4 seem to be the exceptions.
> 
>     * so there is no way for user to change favorite status, and "Switch Profile" always contains one item: "Default Profile". Apparently, it is confusing.
> 
> 
> Personally I don't want to add it into the context menu, because it is really a global instead of  context related action. But currently konsolepart only has a context menu, so put it there for now.
> 
> If the consensus is NOT getting it back, then it is better to remove those related code.
> 
> 
> Diffs
> -----
> 
>   desktop/partui.rc 2eacc48 
>   src/Part.h 31fff53 
>   src/Part.cpp 7bce172 
> 
> Diff: http://git.reviewboard.kde.org/r/104361/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> before
>   http://git.reviewboard.kde.org/r/104361/s/485/
> after
>   http://git.reviewboard.kde.org/r/104361/s/486/
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120328/333799e6/attachment.html>