----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106755/#review20224 -----------------------------------------------------------
Screenshot: Tooltip with mounted device <http://git.reviewboard.kde.org//r/106755/#scomment101> This tooltip looks really odd and out of place this way.:/ - Sebastian Kügler On Oct. 12, 2012, 9:41 a.m., Jonathan Marten wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106755/ > ----------------------------------------------------------- > > (Updated Oct. 12, 2012, 9:41 a.m.) > > > Review request for KDE Base Apps and Plasma. > > > Description > ------- > > If a removable device is mounted using the Plasma device notifier, there is > no indication of what the Unix name of the device is or where it is mounted. > This information may be useful to the user for (a) accessing the mounted > device from non-KDE applications, or (b) troubleshooting mounting or > unmounting problems. > > The attached patch shows this information when the device is hovered over, > just above the "N actions for this device" text. Depending on whether or not > the device is mounted, there are three possibilities that can be shown here: > > /dev/XXX when not mounted > /dev/XXX mounted on /media/YYYY when mounted > /dev/XXX mounted if mounted but the mount point is not > available > > Please be gentle, this is my first QML patch :-) > > > This addresses bug 196939. > http://bugs.kde.org/show_bug.cgi?id=196939 > > > Diffs > ----- > > plasma/generic/applets/devicenotifier/package/contents/ui/DeviceItem.qml > 2a9b3f7 > > Diff: http://git.reviewboard.kde.org/r/106755/diff/ > > > Testing > ------- > > Built kde-workspace with this change, observed operation and display of > device notifier with a selection of removable devices. > > > Screenshots > ----------- > > Device notifier with mounted device > http://git.reviewboard.kde.org/r/106755/s/756/ > Tooltip with mounted device > http://git.reviewboard.kde.org/r/106755/s/769/ > > > Thanks, > > Jonathan Marten > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel