----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122464/#review75564 -----------------------------------------------------------
Ship it! Ship It! - David Edmundson On Feb. 7, 2015, 12:55 p.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, 12:55 p.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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
