Review Request 122464: Remove bell kcm

[Jeremy Whiting]

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

Ship it!


Ship It!

- Jeremy Whiting


On Feb. 7, 2015, 5:55 a.m., Frederik Gladhorn wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122464/
> -----------------------------------------------------------
> 
> (Updated Feb. 7, 2015, 5:55 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> The kcm sets up the xbell, but konsole for example doesn't use the 
> xbell but knotify. The code is also xcb only, I have no idea if it could
> be ported to wayland. For users the knotify way of treating the bell 
> (visual/sound/custom executable) should satisfy all needs.
> 
> Then there is also the accessibility kcm allowing to switch on and off 
> the system bell (using xbell as well) and allowing to flash the screen 
> in a color (I could not get that one to work, neither using xterm nor konsole).
> 
> A recent discussion on the accessibility mailing list also gave no real use case for this duplicate kcm to exist.
> 
> http://lists.kde.org/?t=142023827300002&r=1&w=2
> 
> 
> Diffs
> -----
> 
>   kcms/CMakeLists.txt 06056cd 
>   kcms/bell/CMakeLists.txt f2ebe19 
>   kcms/bell/Messages.sh cb6a478 
>   kcms/bell/bell.cpp 1c4f552 
>   kcms/bell/bell.desktop f1cd552 
>   kcms/bell/bell.h fda09b5 
> 
> Diff: https://git.reviewboard.kde.org/r/122464/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Frederik Gladhorn
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150207/34f4ff99/attachment.html>