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