<table><tr><td style="">Zren created this revision.<br />Zren added a reviewer: Plasma.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />Zren requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11082">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Was just wondering how difficult it would be to implement this. This was practice than anything else, but if you thinks it's worth cleaning up I can do so. Otherwise, reject it.</p>

<p><a href="https://phabricator.kde.org/F5742990" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5742990: 2018-03-06___01-30-43.png</a></p>

<p><a href="https://phabricator.kde.org/F5742991" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5742991: 2018-03-06___01-32-11.png</a></p>

<p>Issues:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">You can filter the streams using a hack that forces PlasmaCore.SortFilterModel to invalidate the filter.</li>
<li class="remarkup-list-item">I duplicated <tt style="background: #ebebeb; font-size: 13px;">BaseMap</tt> in <tt style="background: #ebebeb; font-size: 13px;">map.h</tt> to use a QString key focused on the <tt style="background: #ebebeb; font-size: 13px;">info.name</tt> property. Code duplication sucks. :/</li>
<li class="remarkup-list-item">Every stream uses the system notifications icon. Would need to play around with that.</li>
<li class="remarkup-list-item">I don't believe the "device dropdown" is correct. I think all my streams are set to a "default device index" so it's not selecting the right dropdown option.</li>
<li class="remarkup-list-item">Double scrollbars sucks, but I'd need to look into why there's a binding loop on <tt style="background: #ebebeb; font-size: 13px;">AppletConfiguration.height</tt>. We need to use ScrollView + ListView so that it only loads 4-5 delegates at a time since I personally had 100+ entries (games) which took a few seconds to load the tab when I removed the filter from the Applications tab.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11082">https://phabricator.kde.org/D11082</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/context.cpp<br />
src/kcm/package/contents/ui/Applications.qml<br />
src/kcm/package/contents/ui/StreamListItem.qml<br />
src/kcm/package/contents/ui/main.qml<br />
src/maps.h<br />
src/streamrestore.cpp<br />
src/streamrestore.h</div></div></div><br /><div><strong>To: </strong>Zren, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>