> On Jan. 11, 2016, 8:12 nachm., Kai Uwe Broulik wrote: > > Someone suggested to style it like this: https://i.imgur.com/nPVhWr9.png > > > > While I'm not a huge fan of that excessive green background I agree that > > perhaps the message should be more prominently displayed, ie. moving back > > to showing the icon next to the message and maybe adding a faint highlight > > behind the item. > > Heiko Tietze wrote: > Better than coloring or higlighting parts of the UI you improve > positioning and spacing of controls. That means in this case to indent the > device information. > > Kai Uwe Broulik wrote: > Indent what? where? > > Heiko Tietze wrote: > Sorry, I was completely wrong. The term 'style' put me to the wrong > track, but actually you wanted to indicate the selection. Well done.
I now made it so the selection rectangle shows up behind the device that shows a message and slowly disappears (3s), I think that works nicely. - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126688/#review90936 ----------------------------------------------------------- On Jan. 11, 2016, 4:19 nachm., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126688/ > ----------------------------------------------------------- > > (Updated Jan. 11, 2016, 4:19 nachm.) > > > Review request for Plasma and KDE Usability. > > > Repository: plasma-workspace > > > Description > ------- > > Instead of showing a detached status bar with the device name, show the > message below the device. > > When removing a device, device notifier will pop up and a "You can now safely > remove this device" message will show up; after 5 seconds the message and the > device will disappear. When trying to unmount a device and it fails, device > notifier will show up with an error message that will stay there until device > notifier is closed or another message appears. > > This review consists of two patches: one for fixing the device notifications > engine's wording (I'm open to better verbalizations) and one for the device > notifier > > > Diffs > ----- > > applets/devicenotifier/package/contents/ui/DeviceItem.qml a410e25 > applets/devicenotifier/package/contents/ui/FullRepresentation.qml 9e9aa25 > applets/devicenotifier/package/contents/ui/StatusBar.qml ba827c3 > applets/devicenotifier/package/contents/ui/devicenotifier.qml 29cdaca > dataengines/devicenotifications/ksolidnotify.h fac8cc1 > dataengines/devicenotifications/ksolidnotify.cpp 3935ba3 > > Diff: https://git.reviewboard.kde.org/r/126688/diff/ > > > Testing > ------- > > There a still a couple of glitches: > - the layout doesn't properly reset at times (eg. the delegate height doesn't > update) or fails to show certain items (also looks like a Qt bug) > - sometimes the delegates suddenly overlap each other beacuse the section > thing gets confused when the one item is already gone in the model (looks > like a Qt bug) > - the "no devices" heading doesn't know that there's still a pseudo-device > there > - it cannot actually highlight the device that was safely removed (it's no > longer part of the model and thus has no index), we could do a hack for this > though > - the message doesn't disappear reliably or spontaneously re-appears > - depending on your screen dpi you sometimes get a black and white Info icon > but the error thing is always red, there's a smaller variant for the former > but not the latter apparently, also it seems we lack a proper "task done" > icon, Oxygen had one > > > File Attachments > ---------------- > > Failed to mount > > https://git.reviewboard.kde.org/media/uploaded/files/2016/01/09/8e7142b2-38a9-4867-a067-0e6cdbcaa076__devicenotifierinlinefeedback1.png > Safely remove > > https://git.reviewboard.kde.org/media/uploaded/files/2016/01/09/a43199cf-36a3-4337-9843-cb7375462c57__devicenotifierinlinefeedback2.png > Emblem icon > > https://git.reviewboard.kde.org/media/uploaded/files/2016/01/10/607af6d4-7533-4b1f-b483-182afe9b4173__devicenotifierinlinefeedback3.png > Emblem-error > > https://git.reviewboard.kde.org/media/uploaded/files/2016/01/11/cf96b627-535d-404a-9330-61b171ebda3b__devicenotifierinlinefeedback4.png > emblem-information > > https://git.reviewboard.kde.org/media/uploaded/files/2016/01/11/1ee27871-65f8-4fc2-ae43-1c31a197578d__devicenotifierinlinefeedback5.png > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
