D19824: In saving single file mode an enter/return press on the KDirOperator triggers slotOk

2019-03-17 Thread Nathaniel Graham
ngraham added reviewers: Frameworks, Dolphin.

REPOSITORY
  R241 KIO

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

To: meven, #frameworks, #dolphin
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D19824: In saving single file mode an enter/return press on the KDirOperator triggers slotOk

2019-03-17 Thread Méven Car
meven updated this revision to Diff 54082.
meven added a comment.


  Add deleted blank line

REPOSITORY
  R241 KIO

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19824?vs=54081=54082

BRANCH
  master

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

AFFECTED FILES
  src/filewidgets/kdiroperator.cpp
  src/filewidgets/kdiroperator.h
  src/filewidgets/kfilewidget.cpp
  src/filewidgets/kfilewidget.h
  tests/kfilewidgettest_gui.cpp

To: meven
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D19824: In saving single file mode an enter/return press on the KDirOperator triggers slotOk

2019-03-17 Thread Méven Car
meven updated this revision to Diff 54081.
meven added a comment.


  Remove blank line

REPOSITORY
  R241 KIO

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19824?vs=54080=54081

BRANCH
  master

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

AFFECTED FILES
  src/filewidgets/kdiroperator.cpp
  src/filewidgets/kdiroperator.h
  src/filewidgets/kfilewidget.cpp
  src/filewidgets/kfilewidget.h
  tests/kfilewidgettest_gui.cpp

To: meven
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D19824: In saving single file mode an enter/return press on the KDirOperator triggers slotOk

2019-03-17 Thread Méven Car
meven created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
meven requested review of this revision.

REVISION SUMMARY
  BUG: 385189
  "Save dialog: Return key only bound to Save button when text field has focus"
  
  Draw inspiration from inspiration from https://phabricator.kde.org/D12538 for 
reference.
  
  First attempt, not necessarly the right one.
  But I think it points out the issue at hand:
  
  - the KDirOperator at the moment does not signal when it received an 
enter/return event when it should.
  - This prevents expected behavior

TEST PLAN
  1 Compile an that uses kio open save dialog
  2 Save a file in the app
  3 choose a filename
  4 click on an place on the left
  5 Hit enter

REPOSITORY
  R241 KIO

BRANCH
  master

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

AFFECTED FILES
  src/filewidgets/kdiroperator.cpp
  src/filewidgets/kdiroperator.h
  src/filewidgets/kfilewidget.cpp
  src/filewidgets/kfilewidget.h

To: meven
Cc: kde-frameworks-devel, michaelh, ngraham, bruns