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

Yoann Laissus yoann.laissus at gmail.com
Fri Apr 22 19:18:31 UTC 2016


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

(Updated avr. 22, 2016, 7:18 après-midi)


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 (updated)
-----

  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/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/20160422/4d460e16/attachment.html>


More information about the Plasma-devel mailing list