> On Sept. 19, 2013, 2:57 p.m., Sebastian Kügler wrote: > > Looks good, thanks for the patch, just fix the whitespace, then it's good > > to go. Do you have a Git account? In that case, please push to KDE/4.11, > > master and frameworks-scratch branches.
I don't have a git account; this is my first review request. (So far, I've always simply posted my patches on bugzilla and other people have pushed them to the repository.) - Benedikt ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112812/#review40353 ----------------------------------------------------------- On Sept. 19, 2013, 3:24 p.m., Benedikt Gollatz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112812/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2013, 3:24 p.m.) > > > Review request for Plasma. > > > Description > ------- > > Use type description sort order in the devicenotifier applet when > non-removable devices are configured to be displayed. This avoids apparently > random sort order (by device engine population timestamp) and unneccessary > ListView sections. The problem becomes apparent if other types of devices > besides simple storage volumes are configured in /etc/fstab, like for example > network mounts. > > Fixes bug #324459. > > > This addresses bug 324459. > http://bugs.kde.org/show_bug.cgi?id=324459 > > > Diffs > ----- > > > plasma/generic/applets/devicenotifier/package/contents/ui/devicenotifier.qml > 9b6132e > > Diff: http://git.reviewboard.kde.org/r/112812/diff/ > > > Testing > ------- > > Works for me using KDE 4.10.5 packaged with Fedora 19. > > > Thanks, > > Benedikt Gollatz > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
