[Marble-devel] Review Request 119391: Added a small 'animation' when merging two nodes

Cruceru Calin-Cristian crucerucalincristian at gmail.com
Mon Jul 21 22:09:42 UTC 2014


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

(Updated July 22, 2014, 1:09 a.m.)


Review request for Marble.


Repository: marble


Description
-------

As the title says, this patch includes changes for adding a small animation when two nodes are being merged. Before, the merging was a little bit too 'fast' and it could have confused the users. I think that this way, the merging is much nicer.

Also, you will see in this patch a couple of other small changes which I decided to make after I discovered a couple of bugs. For example one could not interact with the map anymore after selecting the second node to be merged because in AnnotatePlugin, at each mouse press, the item got selected for moving and deselected at mouse release, but when selecting the second node to be merged, the release event was not caught by the item.

I need your feedback regarding the way the new feature behaves :).


Diffs
-----

  src/plugins/render/annotate/AreaAnnotation.cpp e0ed211 
  src/plugins/render/annotate/CMakeLists.txt 4792a78 
  src/plugins/render/annotate/EditPolygonDialog.cpp cc25874 
  src/plugins/render/annotate/MergingNodesAnimation.h PRE-CREATION 
  src/plugins/render/annotate/MergingNodesAnimation.cpp PRE-CREATION 
  src/plugins/render/annotate/AnnotatePlugin.h 6bb9b61 
  src/plugins/render/annotate/AnnotatePlugin.cpp dd403e6 
  src/plugins/render/annotate/AreaAnnotation.h 6c305fe 

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


Testing
-------


Thanks,

Cruceru Calin-Cristian

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


More information about the Marble-devel mailing list