<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: VDG, Plasma.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ngraham 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/D18528">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When the Desktop Toolbox is dragged out of the corner, its label is shown. Currently this label is not very clear since it consists only of the name of the current activitym which by default is "Default".</p>

<p>This patch introduces three changes:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">When the label is shown, always prepend it with the widget's actual name: "Desktop Toolbox"</li>
<li class="remarkup-list-item">Append the word "Activity" so it's obvious what the preceeding string refers to</li>
<li class="remarkup-list-item">Don't show the activity name if it's "Default" since this signifies that the user does not use Activities and would be confused by seeing references to it</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>There are no visible changes to the default appearance. The Desktop Toolbox still has no label when it's in a corner. When dragged out of the corner, here's how it looks with this patch:</p>

<p>With the Default activity: <a href="https://phabricator.kde.org/F6567804" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6567804: Desktop Toolbox, default activity</a></p>

<p>With a differently-named activity: <a href="https://phabricator.kde.org/F6567805" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6567805: Desktop toolbox, differently-named activity</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>desktop-toolbox-clearer-naming (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18528">https://phabricator.kde.org/D18528</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml</div></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>