Review Request 113425: the map and divisions are added to the scene handled by the graphicsview.

Jaime Torres Amate jtamate at gmail.com
Fri Oct 25 16:19:07 UTC 2013


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

(Updated Oct. 25, 2013, 4:19 p.m.)


Review request for KDE Edu and Albert Astals Cid.


Changes
-------

second version. 
Solves a crash not detected before for setting setMouseTracking(true) too early.
Hides the cursor (to be more like before) and hide/show the division when zooming or moving.
(the 3 remaining spaces removed after seeing them in the review).


Bugs: 285330
    http://bugs.kde.org/show_bug.cgi?id=285330


Repository: kgeography


Description
-------

Using addPixmap to a QGraphicsView, just let the view handle the zoom, moves.... Just handling all the view events to move the division smoothly.
This time, just paint the background white and then draw the map without divisions (before the first division is asked), then add the divisions as they are asked.
(as always, the red spaces will be gone). The variable "punto" will be translated.


Diffs (updated)
-----

  src/placeasker.cpp 07bea6c 
  src/placemapwidget.h 6905a14 
  src/placemapwidget.cpp 94a4460 

Diff: http://git.reviewboard.kde.org/r/113425/diff/


Testing
-------

Zoom, move (it behaves like before), with several maps in two computers (one that was not affected by the bug and one that was affected).


Thanks,

Jaime Torres Amate

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20131025/8b6cf78d/attachment.html>


More information about the kde-edu mailing list