Review Request 127710: Some cleanup in the applet and the KCM

Yoann Laissus yoann.laissus at gmail.com
Sat Apr 23 11:16:04 UTC 2016



> On avr. 23, 2016, 10:51 matin, David Rosca wrote:
> > Actually, it should use TextMetrics and not FontMetrics, sorry. Can you please change it and then commit?
> 
> David Rosca wrote:
>     Or FontMetrics but with boundingRect(string).width and not advanceWidth, advanceWidth is too big.

I don't see a lot of differences between those two properties.
Does advanceWidth include some margins ?

I'll do another commit for that as I already commited with FontMetrics.


- Yoann


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127710/#review94776
-----------------------------------------------------------


On avr. 23, 2016, 11:04 matin, Yoann Laissus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127710/
> -----------------------------------------------------------
> 
> (Updated avr. 23, 2016, 11:04 matin)
> 
> 
> Review request for Plasma and David Rosca.
> 
> 
> Repository: plasma-pa
> 
> 
> Description
> -------
> 
> Branche "cleanup" here : git://anongit.kde.org/clones/plasma-pa/laissus/plasma-pa.git
> 
> 1. Move sourceIndex and sinkIndex properties to a generic one called deviceIndex in the Stream class.
> 
> It helps us to remove the deviceType checks in the KCM (DeviceComboBox)
> 
> 2. Drop sub component support for list items of the applet as it's not used anymore
> 
> As we discussed in a previous RR, it's not planned to use this component so let's clean this up !
> I also unified source and sink list items into a generic one.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt a40b99623fbbe2ba3fd874a0f73dd218fc94a42d 
>   applet/contents/ui/DeviceListItemBase.qml 3376c73030470f27a11f0bbedd436fed2676fad5 
>   applet/contents/ui/ListItemBase.qml d77d25a51b09149a1cf479eec0053184ae8625e6 
>   applet/contents/ui/RootItem2.qml afcae47d2a9ebfa8f442e9cda93cbf53ac238ec9 
>   applet/contents/ui/SinkListItem.qml 1f9c5df28405d1e5ef65deb2325d293ecb4884b1 
>   applet/contents/ui/SourceListItem.qml de02c3845b4ab794c560da37c8bfe23c6eff8e46 
>   applet/contents/ui/main.qml a8dbf7ded667acc6c830f70bf542b38020beaaf8 
>   src/kcm/package/contents/ui/DeviceComboBox.qml e5a87e733d3009c9f4d08623f65a1c8eede303b7 
>   src/kcm/package/contents/ui/SinkInputView.qml 6e165ad5349129bb10cbae39c0f8300308b44dc6 
>   src/kcm/package/contents/ui/SourceOutputView.qml 3cd978b4bbf8c53e02e098e73d33ed1a169cf0c6 
>   src/kcm/package/contents/ui/StreamListItem.qml b12e3f3e344fea57cc08085eb68390bac3c11749 
>   src/kcm/package/contents/ui/VolumeSlider.qml 7904a5df7dcecd4abbe577d08ced131c607cbc1d 
>   src/sinkinput.h a9a368cc34fcfd42079f064d90f09f52690c5fca 
>   src/sinkinput.cpp cc5283b71c9af76dcc678644bd3d282bd3473f8a 
>   src/sourceoutput.h 12f4a6c1d5666aea7653765b75aa099e8c1fe422 
>   src/sourceoutput.cpp 66152869618426d8fb27f60d4722816ffbe5144d 
>   src/stream.h 109997c1b3160bb1752d874db218d7f5a3e55f47 
>   src/stream.cpp 55c0c6bf40d7b3d8ba3e97123e9f1eec4c0c9230 
> 
> Diff: https://git.reviewboard.kde.org/r/127710/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Yoann Laissus
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160423/77f9afa8/attachment.html>


More information about the Plasma-devel mailing list