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 18:41:30 UTC 2013


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

(Updated Oct. 25, 2013, 6:41 p.m.)


Review request for KDE Edu and Albert Astals Cid.


Changes
-------

@Albert, can you break it with the third version? I can not with the second (even running under valgrind).
removed p_firstShow.


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.h 14dce46 
  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/d681c5e7/attachment.html>


More information about the kde-edu mailing list