<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: VDG, Plasma, hein, davidedmundson.<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/D26884">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Currently there's a top-level ColorScope that is almost entirely unused except for<br />
introducing a bug whereby the label text in the pop-up is the wrong color. Removing the<br />
ColorScope fixes the bug and does not change the appearance of anything else anywhere else.</p>

<p>BUG: 416658<br />
FIXED-IN: 5.18.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Hover over a folder and click on button to open a folder pop-up <br />
See that the text color is correct now: <a href="https://phabricator.kde.org/F7925905" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7925905: After.png</a></p>

<p>Use Folder view as a containment, as a widget, and in a panel<br />
See that text color is fine everywhere</p>

<p>Use a bunch of different color schemes and repeat the above tests and see that everything<br />
still looks good</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>fix-folderview-popup-text-color (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26884">https://phabricator.kde.org/D26884</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/FolderViewLayer.qml</div></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma, hein, davidedmundson<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>