[prev in list] [next in list] [prev in thread] [next in thread] 

List:       calligra-devel
Subject:    Re: Need review and testing for 'zoom-pan-testing-kazakov' branch
From:       Boudewijn Rempt <boud () valdyas ! org>
Date:       2012-09-03 8:25:05
Message-ID: 201209031025.05670.boud () valdyas ! org
[Download RAW message or body]

On Sunday 02 September 2012 Sep, Dmitry Kazakov wrote:
> Hi!
> 
> I've just finished the branch which fixes various zoom/pan bugs in Krita
> and adds comprehensive testing for it. It would be really cool if someone
> could test it and review before I merge it to master. I send the mail to
> calligra list as well, because it touches a couple of classes in
> ./libs/flake/ as well.
> 
> This branch touches KoCanvasControllerWidget, so, theoretically, it may
> affect other applications, so it might be necessary to check them for
> regressions as well. I checked Karbon and it seems to work fine.
> 
> What should be tested in Krita:
> 1) Usual Zoom and Pan
> 2) Zoom and Pan in Fullscreen mode
> 3) Zoom and Pan on rotated canvas
> 4) Zoom and Pan on mirrored canvas
> 5) (optional) Zoom and Pan with VastScrolling disabled.
> 
> In other applications:
> 1) There should be no regressions ;)
> 
> 
> Thank you in advance!
> 
> 
> PS:
> 
> Small howto:
> The changes are in the branch 'zoom-pan-testing-kazakov'. So to check you
> need to switch to it:
> 
> git fetch
> git checkout origin/zoom-pan-testing-kazakov
> <build your Krita>
> 
> To switch back to master:
> 
> git checkout master
> 

Hm, now I get another compile error:

[ 76%] Building CXX object \
                karbon/ui/CMakeFiles/karbonui.dir/KarbonZoomController.cpp.o
/home/boud/kde/src/calligra/karbon/ui/KarbonZoomController.cpp: In member function \
                ‘void KarbonZoomController::setZoom(KoZoomMode::Mode, qreal)':
/home/boud/kde/src/calligra/karbon/ui/KarbonZoomController.cpp:131:54: error: \
‘documentRect' was not declared in this scope make[2]: *** \
[karbon/ui/CMakeFiles/karbonui.dir/KarbonZoomController.cpp.o] Error 1 make[1]: *** \
[karbon/ui/CMakeFiles/karbonui.dir/all] Error 2


-- 
Boudewijn Rempt
http://www.valdyas.org, http://www.krita.org, http://www.boudewijnrempt.nl
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic