[Marble-devel] Review Request 123221: Annotate plugin: EditPolylineDialog and EditPolygonDialog fix.

Torsten Rahn tackat at kde.org
Thu Apr 2 18:42:27 UTC 2015



> On April 2, 2015, 6:42 nachm., Torsten Rahn wrote:
> > src/plugins/render/annotate/EditPolygonDialog.cpp, line 99
> > <https://git.reviewboard.kde.org/r/123221/diff/1/?file=359751#file359751line99>
> >
> >     hm, why not connect it to updatePolygon() directly? You can connect slots with a certain parameter to one which loses that parameter (you just can not connect it to one with an additional parameter).

Works great apart from that :)


- Torsten


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


On April 2, 2015, 10:39 vorm., Constantin Mihalache wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123221/
> -----------------------------------------------------------
> 
> (Updated April 2, 2015, 10:39 vorm.)
> 
> 
> Review request for Marble, Calin Cruceru and Torsten Rahn.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> Bugs:
> 
> #1 In EditPolylineDialog, modifying the width spinbox didn't update the polyline on the globe.
> 
> #2 In EditPolygonDialog, modifying the width spinbox didn't update the polyline on the globe.
> 
> 
> Diffs
> -----
> 
>   src/plugins/render/annotate/EditPolygonDialog.h 0dbb089 
>   src/plugins/render/annotate/EditPolygonDialog.cpp 84e5697 
>   src/plugins/render/annotate/EditPolylineDialog.h 4d383b8 
>   src/plugins/render/annotate/EditPolylineDialog.cpp a06d55d 
> 
> Diff: https://git.reviewboard.kde.org/r/123221/diff/
> 
> 
> Testing
> -------
> 
> It works for me.
> 
> 
> Thanks,
> 
> Constantin Mihalache
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150402/4b5b6250/attachment-0001.html>


More information about the Marble-devel mailing list