----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119448/#review63095 -----------------------------------------------------------
There's no change in zoom yet. This is because centerOn() is connected directly to MarbleWidget's centerOn() slot which ignores any altitude. See TourWidget::centerOn() for a working implementation. When the route is changed, any running/existing tour and tour playback should be stopped and deleted. Currently you can change the route and the old tour will still be played, following the already gone route. src/lib/marble/routing/RoutingWidget.cpp <https://git.reviewboard.kde.org/r/119448/#comment43841> Please remove, causes a wrong icon on the very first play click. - Dennis Nienhüser On July 24, 2014, 3:25 p.m., Sanjiban Bairagya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119448/ > ----------------------------------------------------------- > > (Updated July 24, 2014, 3:25 p.m.) > > > Review request for Marble. > > > Repository: marble > > > Description > ------- > > This patch helps to view a route as a tour, by clicking on a toggled > Play/Pause button in the RoutingWidget when a route has been loaded > > > Diffs > ----- > > src/lib/marble/TourPlayback.cpp 6682322 > src/lib/marble/routing/RoutingWidget.h b0e6b6d > src/lib/marble/routing/RoutingWidget.cpp 7abee97 > > Diff: https://git.reviewboard.kde.org/r/119448/diff/ > > > Testing > ------- > > > Thanks, > > Sanjiban Bairagya > >
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
