ngraham added inline comments.

INLINE COMMENTS

> ClipboardPage.qml:81
>          Layout.fillWidth: true
> -        visible: !emptyHint.visible
> +        visible: clipboardMenu.model.count > 0
>  

Can this be `enabled:` rather than `visible:`? That's what I did in D29132 
<https://phabricator.kde.org/D29132> and I think it's nicer and more consistent 
with the mockups.

REPOSITORY
  R120 Plasma Workspace

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

To: ngraham, #vdg, #plasma, broulik, niccolove, cblack
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to