ngraham added inline comments.

INLINE COMMENTS

> Preview.qml:135
> +            }
> +            QQC2.ToolButton {
> +                visible: !iconPreview.screenshotting

There's plenty of space for a button that has visible text. I'd recommend doing 
that instead of using a tiny icons-only button.

REPOSITORY
  R118 Plasma SDK

REVISION DETAIL
  https://phabricator.kde.org/D23712

To: cblack, #vdg
Cc: ndavis, filipf, davidedmundson, ngraham, plasma-devel, #vdg, LeGast00n, 
The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to