D8598: FolderView: position files at drop event target position

Andras Mantia noreply at phabricator.kde.org
Fri Nov 3 14:47:07 UTC 2017


amantia added a comment.


  I didn't tested the code yet, but in general looks good for me.

INLINE COMMENTS

> foldermodel.cpp:181
> +        if (!m_dropTargetPositions.isEmpty()) {
> +            qDebug() << "clearing drop target positions after timeout:" << m_dropTargetPositions;
> +            m_dropTargetPositions.clear();

Either remove or use categorized logging

REPOSITORY
  R119 Plasma Desktop

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

To: mwolff, hein, amantia
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171103/200e2493/attachment.html>


More information about the Plasma-devel mailing list