[Marble-devel] Review Request 118495: Marble input handling refactor. Support for mouse, touch and pinch in MarbleQuickItem.

Adam Dabrowski adamdbrw at gmail.com
Thu Jun 5 07:30:14 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118495/
-----------------------------------------------------------

(Updated June 5, 2014, 7:30 a.m.)


Review request for Marble and Dennis Nienhüser.


Changes
-------

Fixed the regression issues and all other from Dennis review


Repository: marble


Description
-------

Functionalities:
- Mouse, touch and pinch input handling for MarbleQuickItem (keyboard was enabled earlier)

Other changes: 
- Refactored input handling to reuse code, broke into smaller pieces (functions)
- Extended the example (marbleQuick2)

Known issues: 
- Some refactoring still should be done around input classes
- Inertial rotation is disabled by default in MarbleQuickItem because of some issues (mostly touch-related). However, issues are rather minor so feel free to enable it back.


Diffs (updated)
-----

  examples/cpp/marbleQuick2/main.cpp 640baa6 
  examples/cpp/marbleQuick2/main.qml 76947ed 
  src/lib/marble/CMakeLists.txt 4add26b 
  src/lib/marble/MarbleAbstractPresenter.h 7c047ed 
  src/lib/marble/MarbleAbstractPresenter.cpp 98855bb 
  src/lib/marble/MarbleInputHandler.h PRE-CREATION 
  src/lib/marble/MarbleInputHandler.cpp PRE-CREATION 
  src/lib/marble/MarbleQuickItem.h 8fd5657 
  src/lib/marble/MarbleQuickItem.cpp 0ccc0ac 
  src/lib/marble/MarbleWidget.cpp acc270a 
  src/lib/marble/MarbleWidgetInputHandler.h f763d1e 
  src/lib/marble/MarbleWidgetInputHandler.cpp 912c12d 

Diff: https://git.reviewboard.kde.org/r/118495/diff/


Testing
-------

- Using these changes since some time in a working app that is based on a MarbleQuickItem subclass (on both unix PC and a touch device)
- Built and run marbleQuick2 example on both PC and a touch device.
- Did good tests for MarbleQuickItem, basic regression testing for MarbleWidget and no testing for Qt4.


Thanks,

Adam Dabrowski

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20140605/a7d63d0d/attachment.html>


More information about the Marble-devel mailing list