[Marble-devel] Review Request 118190: Functionality for Edit buttons of FlyTo and TourControl items of Tour Playlist

[Dennis Nienhüser]

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


Can you please check the code style, especially the spacing near round brackets?



src/lib/marble/TourWidget.h
<https://git.reviewboard.kde.org/r/118190/#comment40425>

    Where is it used?



src/lib/marble/TourWidget.cpp
<https://git.reviewboard.kde.org/r/118190/#comment40427>

    Please remove



src/lib/marble/TourWidget.cpp
<https://git.reviewboard.kde.org/r/118190/#comment40426>

    Please remove


- Dennis Nienhüser


On May 17, 2014, 11:37 p.m., Sanjiban Bairagya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118190/
> -----------------------------------------------------------
> 
> (Updated May 17, 2014, 11:37 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> This patch implements the following unctionalities:
> - when an user clicks on the Edit button of a FlyTo element, then it will prompt the user to make the map's focus point the new point which he/she wants to make the new coordinates of the element. Clicking on Edit again will update and save this new coordinate.
> - when an user clicks on the Edit button of a TourControl element, it will show two radio buttons, and the user has select one of them. When he clicks on any one of "Play" or "Pause", the radio buttons disappear and the changes get reflected on the playlist immediately. The "Edit" button in this case is a toggle button between Change/Not-Change
> 
> 
> Diffs
> -----
> 
>   src/lib/marble/TourWidget.h c38d620 
>   src/lib/marble/TourWidget.cpp aff7e99 
> 
> Diff: https://git.reviewboard.kde.org/r/118190/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sanjiban Bairagya
> 
>

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