Review Request: Use Product instead of description for device names

Aaron J. Seigo aseigo at kde.org
Sun Sep 30 09:27:26 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106637/#review19631
-----------------------------------------------------------

Ship it!


very sensible and nice improvement. as long as the product name is always sth sensible and there (e.g. never an empty string) then this looks good; if the product can be an empty string, then it should fall back to the hotplug text (perhaps in the DataEngine itself)

- Aaron J. Seigo


On Sept. 29, 2012, 8:21 p.m., Alex Fiestas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106637/
> -----------------------------------------------------------
> 
> (Updated Sept. 29, 2012, 8:21 p.m.)
> 
> 
> Review request for Plasma and Viranch Mehta.
> 
> 
> Description
> -------
> 
> The description is useful for when the device is not hotpluggable/removeable, for example to show:
> 96.3 GiB Hard Drive
> 15.1 GiB Hard Drive
> 
> instead of two identical labels.
> 
> But when it comes to removable/hotpluggable we want to show the Product to be able to show:
> 
> Nokia N9
> Nexus 7
> 
> Instead of 
> Portable Media Player
> Portable Media Player
> 
> The difference can be shown also in the attached screenshots.
> 
> 
> Diffs
> -----
> 
>   plasma/generic/applets/devicenotifier/package/contents/ui/devicenotifier.qml b23df45 
> 
> Diff: http://git.reviewboard.kde.org/r/106637/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> before
>   http://git.reviewboard.kde.org/r/106637/s/743/
> after
>   http://git.reviewboard.kde.org/r/106637/s/744/
> 
> 
> Thanks,
> 
> Alex Fiestas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120930/9180730b/attachment.html>


More information about the Plasma-devel mailing list