webbrowser-app

Code review comment for lp:~ahayzen/webbrowser-app/dnd-handle-design-001

  1. dnd-handle-design-001
  2. Merge into staging
Revision history for this message
Olivier Tilloy (osomon) wrote :

In setPreviewBorderWidth(), previewBorderWidthChanged should be emitted, not previewTopCropChanged.

In drag-helper.cpp, the include of <QtCore/QSizeF> is not needed, as it’s already included in drag-helper.h. Same for <QtGui/QColor>.

review: Needs Fixing

« Back to merge proposal