Review Request 122464: Remove bell kcm

[Frederik Gladhorn]

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

(Updated Feb. 8, 2015, 8:13 p.m.)


Status
------

This change has been marked as submitted.


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/20150208/f909e4ac/attachment.html>