Review Request 122698: Kexi maps choosing Map Theme Id.

[Radosław Wicik]

> On Lut 24, 2015, 11:55 rano, Jarosław Staniek wrote:
> > plugins/reporting/maps/MapRenderer.cpp, line 72
> > <https://git.reviewboard.kde.org/r/122698/diff/1/?file=351234#file351234line72>
> >
> >     Why here and not in MapRenderer()?

Some themes enable overview map after setting them up, this is why we need to disable it each time we set theme.


- Radosław


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


On Mar 4, 2015, 11:44 po południu, Radosław Wicik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122698/
> -----------------------------------------------------------
> 
> (Updated Mar 4, 2015, 11:44 po południu)
> 
> 
> Review request for Calligra, Andrius da Costa Ribas, Oleg Kukharchuk, Adam Pigg, Jarosław Staniek, Dimitrios Tanis, and Tomasz Olszak.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> Kexi maps choosing Map Theme Id in reports.
> 
> 
> Diffs
> -----
> 
>   plugins/reporting/maps/KoReportItemMaps.h 62eb15db4ef89502a1edbba6ee49cf85dd6521e9 
>   plugins/reporting/maps/KoReportItemMaps.cpp c29c0c51d17db8ce3f0c7e672e4d098e40a78f59 
>   plugins/reporting/maps/MapRenderer.cpp 687179c7fe4b1ae70fc31ab0566eab33fe6f7630 
>   plugins/reporting/maps/KoReportDesignerItemMaps.cpp d76b259720955304de9dd1f7819f9d2b0f630f2c 
> 
> Diff: https://git.reviewboard.kde.org/r/122698/diff/
> 
> 
> Testing
> -------
> 
> Maunual testing
> 
> 
> Thanks,
> 
> Radosław Wicik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20150304/3acfa3ec/attachment.htm>