Review Request 109622: Move "Bluetooth" entry from Places to Devices category

[Frank Reininghaus]

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

Ship it!


Thanks for the patch, looks reasonable!

Even though the places view in kdelibs does not support categories, maybe the "device/non-device" status could affect the default sorting of the places? Speaking of sorting: could you check before pushing if it still works OK in the file dialog's places view after the sorting of the devices is changed in Dolphin and vice versa? Can't test it myself due to lack of suitable hardware.



- Frank Reininghaus


On March 20, 2013, 10:11 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109622/
> -----------------------------------------------------------
> 
> (Updated March 20, 2013, 10:11 p.m.)
> 
> 
> Review request for Dolphin, Àlex Fiestas and Frank Reininghaus.
> 
> 
> Description
> -------
> 
> This patch moves the "Bluetooth" entry from Places to Devices category.
> Made sense to me that Bluetooth stuff is also under devices. Also makes Bluetooth devices (ie. you add your smartphone to the places list) appear there.
> 
> 
> This addresses bug 301276.
>     http://bugs.kde.org/show_bug.cgi?id=301276
> 
> 
> Diffs
> -----
> 
>   dolphin/src/panels/places/placesitem.cpp 75e14d0 
> 
> Diff: http://git.reviewboard.kde.org/r/109622/diff/
> 
> 
> Testing
> -------
> 
> Works as expected.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130321/269eba0e/attachment.htm>