<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, Plasma: Design.<br />broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.</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/D2691" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Use the compact representation for indicating status</li>
<li class="remarkup-list-item">In the expanded representation just place a button</li>
<li class="remarkup-list-item">Use declarative bindings as much as possible</li>
</ul>

<p>BUG: 368192<br />
FIXED-IN: 5.8.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I just wanted to fix the icon but then decided to give it a major cleanup.</p>

<p>Tested:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Enabling / disabling</li>
<li class="remarkup-list-item">Disabling with no external mouse prompts for confirmation</li>
<li class="remarkup-list-item">With no touchpad the icon turns gray (and the applet should just hide completely but doesn't work in tray, see below)</li>
</ul>

<p>I stumbled upon two bugs in system tray:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">When I have an applet with preferredRepresentation being expandedRepresentation systray tries to be smart and then expands on plasmashell startup and shows an empty spot for fthe applet</li>
<li class="remarkup-list-item">HiddenStatus applets are still shown in the popup list like PassiveStatus ones</li>
</ul>

<p><a href="https://phabricator.kde.org/F278196" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F278196: Screenshot_20160907_114941.png</a></p>

<p>Expanded representation in system tray, this is only accessible when in the popup and clicking the text of the applet instead of its icon. In all other cases clicking the icon just turns it on/off.<br />
<a href="https://phabricator.kde.org/F278197" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F278197: Screenshot_20160907_115011.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2691" rel="noreferrer">https://phabricator.kde.org/D2691</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/touchpad/src/applet/qml/contents/ui/touchpad.qml</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, Plasma: Design<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>