kdautovic added inline comments.

INLINE COMMENTS

> davidedmundson wrote in klipper.cpp:744
>   bIgnoreImages || !data-> ...

But bIgnoreImages is used later in the code, when determining whether to save 
images to history or not. Is it really necessary here? If it's added here then 
we're back to square one because copying screenshots to clipboard won't work 
with the default behavior of Klipper.

REPOSITORY
  R120 Plasma Workspace

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

To: kdautovic, #plasma_workspaces, davidedmundson
Cc: ngraham, davidedmundson, plasma-devel, #plasma_workspaces, jraleigh, GB_2, 
ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart

Reply via email to