ahmadsamir added inline comments.

INLINE COMMENTS

> kfilewidget.cpp:1913
> +    } else if (watched == d->ops && event->type() == QEvent::KeyPress) {
> +        if (keyEvent->key() == Qt::Key_Return) {
> +            // ignore return events from the KDirOperator

I would combine the two if's (less nesting); and Qt::Key_Enter should be 
handled here too?

REPOSITORY
  R241 KIO

BRANCH
  arcpatch-D27455

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

To: meven, dfaure, ngraham, #frameworks
Cc: ahmadsamir, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, 
ngraham, bruns

Reply via email to