----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128832/#review98898 -----------------------------------------------------------
Fix it, then Ship it! applet/contents/ui/ConnectionItem.qml (line 310) <https://git.reviewboard.kde.org/r/128832/#comment66584> Make property readonly and add an empty line afterwards Also perhaps add a comment on how Math.max in conjunction with concatenated QList<qreal> doesn't work - Kai Uwe Broulik On Sept. 5, 2016, 10:58 vorm., Jan Grulich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128832/ > ----------------------------------------------------------- > > (Updated Sept. 5, 2016, 10:58 vorm.) > > > Review request for Network Management and Plasma. > > > Bugs: 368247 > http://bugs.kde.org/show_bug.cgi?id=368247 > > > Repository: plasma-nm > > > Description > ------- > > When the upload speed is higher then the download we should set maximum > plotter value based on that. > > > Diffs > ----- > > applet/contents/ui/ConnectionItem.qml 3440e39 > > Diff: https://git.reviewboard.kde.org/r/128832/diff/ > > > Testing > ------- > > > Thanks, > > Jan Grulich > >
