----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115712/#review49692 -----------------------------------------------------------
src/shell/osd.cpp <https://git.reviewboard.kde.org/r/115712/#comment34968> First thing - coding style dictates {} even for oneliners ;) Then - this will not show the OSD on the keypress and thus will not provide any feedback to the user, giving the impression of something broken. If it's full, we want to show it's full. So let's cap the value, you can use qBound for that. - Martin Klapetek On Feb. 13, 2014, 7:51 a.m., Bhushan Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115712/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2014, 7:51 a.m.) > > > Review request for Plasma and Martin Klapetek. > > > Repository: plasma-framework > > > Description > ------- > > $summary > > Actually this will never happen as OSD will be triggered by applications only > but better to check it. > > > Diffs > ----- > > src/shell/osd.cpp e94d225 > > Diff: https://git.reviewboard.kde.org/r/115712/diff/ > > > Testing > ------- > > qdbus org.kde.plasma_shell /org/kde/osdService > org.kde.osdService.volumeChanged 203 > qdbus org.kde.plasma_shell /org/kde/osdService > org.kde.osdService.volumeChanged -40 > > OSD does not show up > > > Thanks, > > Bhushan Shah > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
