Review Request 128434: rename some kcm desktop files
Thomas Pfeiffer
thomas.pfeiffer at kde.org
Wed Jul 13 12:04:16 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128434/#review97343
-----------------------------------------------------------
See the inline comments. The things I've +1'd can be shipped from my perspective
kcms/baloo/kcm_baloofile.desktop (line 15)
<https://git.reviewboard.kde.org/r/128434/#comment65707>
Hm, I'm not sure. From a user's perspective, it's relevant where the search can find things, "indexing" is more of an implementation detail for the user.
kcms/keys/keys.desktop (line 14)
<https://git.reviewboard.kde.org/r/128434/#comment65708>
I thought we wanted to have both in the same KCM, just under different headings?
We should only change that if we actually decide to split all application shortcuts out into a separate KCM.
kcms/ksmserver/kcmsmserver.desktop (line 3)
<https://git.reviewboard.kde.org/r/128434/#comment65709>
+1
kcms/phonon/kcm_phonon.desktop (line 13)
<https://git.reviewboard.kde.org/r/128434/#comment65710>
It does also contain settings for video recording devices, so it is not just audio.
kcms/standard_actions/standard_actions.desktop (line 15)
<https://git.reviewboard.kde.org/r/128434/#comment65711>
+1
kcms/translations/translations.desktop (line 14)
<https://git.reviewboard.kde.org/r/128434/#comment65712>
+1
- Thomas Pfeiffer
On July 13, 2016, 8:26 a.m., Andreas Kainz wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128434/
> -----------------------------------------------------------
>
> (Updated July 13, 2016, 8:26 a.m.)
>
>
> Review request for Plasma.
>
>
> Repository: plasma-desktop
>
>
> Description
> -------
>
> change see in the diff
>
>
> Diffs
> -----
>
> kcms/baloo/kcm_baloofile.desktop 919fc8e
> kcms/desktoppaths/desktoppath.desktop 2567c76
> kcms/keys/keys.desktop ac6afb6
> kcms/ksmserver/kcmsmserver.desktop 8ed358b
> kcms/phonon/kcm_phonon.desktop ed804a3
> kcms/standard_actions/standard_actions.desktop a454dbd
> kcms/translations/translations.desktop daf9469
>
> Diff: https://git.reviewboard.kde.org/r/128434/diff/
>
>
> Testing
> -------
>
>
> File Attachments
> ----------------
>
> adjustments
> https://git.reviewboard.kde.org/media/uploaded/files/2016/07/13/ac3e5c78-9bb4-4e80-a5ce-0e4dfb38ae22__SystemSettings.png
>
>
> Thanks,
>
> Andreas Kainz
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160713/7a9e1f84/attachment.html>
More information about the Plasma-devel
mailing list