[Marble-devel] Review Request 124297: Some qml properties for MarbleQuickItem

[Gábor Péterffy]

> On júl. 9, 2015, 7:47 du, Dennis Nienhüser wrote:
> > Can you please have a look at these patches as well? I think we should commit them asap and base your work on top of them.
> > https://git.reviewboard.kde.org/r/123291/
> > https://git.reviewboard.kde.org/r/123202/
> > https://git.reviewboard.kde.org/r/123204/

I have reviewed them. The first two is okay in my opinion, but I had something to say to the third one. Please view it!


- Gábor


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


On júl. 8, 2015, 6:17 du, Gábor Péterffy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124297/
> -----------------------------------------------------------
> 
> (Updated júl. 8, 2015, 6:17 du)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> These are added to solve the todo in the example.
> 
>         map()->setShowFrameRate(false);
>         map()->setProjection(Spherical);
>         map()->setMapThemeId("earth/openstreetmap/openstreetmap.dgml");
>         map()->setShowAtmosphere(false);
>         map()->setShowCompass(false);
>         map()->setShowClouds(false);
>         map()->setShowCrosshairs(false);
>         map()->setShowGrid(false);
>         map()->setShowOverviewMap(false);
>         map()->setShowOtherPlaces(false);
>         map()->setShowScaleBar(false);
>         map()->setShowBackground(false);
>         
>   The modification of the example will be in the following patch.
> 
> 
> Diffs
> -----
> 
>   src/lib/marble/MarbleQuickItem.cpp 94a0a1a498d4be29778f055d6c05bb532f12ee44 
>   src/lib/marble/MarbleQuickItem.h 41e20bb69bbeb9c1d662cd943ecb13674fd04498 
>   src/lib/marble/MarbleGlobal.h cf2768f16c1427ea7f465391827666261b806b80 
>   src/lib/marble/EnumWrappers.cpp PRE-CREATION 
>   src/lib/marble/EnumWrappers.h PRE-CREATION 
>   src/lib/marble/CMakeLists.txt 8b36d0843f585c09c62bf044d0cfb0651a13d4b0 
> 
> Diff: https://git.reviewboard.kde.org/r/124297/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gábor Péterffy
> 
>

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