----------------------------------------------------------- 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 > >
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
