Review Request 128808: workspace option kcm rename to tooltip

Andreas Kainz kainz.a at gmail.com
Sat Sep 3 10:30:40 UTC 2016



> On Sept. 1, 2016, 4:59 vorm., Kai Uwe Broulik wrote:
> > -1
> > 
> > I was actually planning to add some workspace options (animation speed, OSD settings, etc) in there.

if you add something, good, but please rename it to something the user can understand what the kcm is for. workspace doesn't meen anything specific the global group is named workspace.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128808/#review98811
-----------------------------------------------------------


On Sept. 1, 2016, 7:07 vorm., Andreas Kainz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128808/
> -----------------------------------------------------------
> 
> (Updated Sept. 1, 2016, 7:07 vorm.)
> 
> 
> Review request for Plasma, KDE Usability and Heiko Tietze.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> the request rename the workspace kcm (workspace -> desktop behavior -> workspace) to tooltip and move it to the notification section. also the description was changed to desktop tooltips.
> 
> I'm not sure if this kcm exists at the survey from heiko but I don't think so it look like a placeholder kcm where only the tooltip selection was available now so, please rename it to the think you can do and when you add more stuff in the future we can rename it back, but now it's a kcm for turn tooltips on/off.
> 
> 
> Diffs
> -----
> 
>   kcms/workspaceoptions/workspaceoptions.desktop ec1b233 
> 
> Diff: https://git.reviewboard.kde.org/r/128808/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andreas Kainz
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160903/91cb64b1/attachment.html>


More information about the Plasma-devel mailing list