----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124622/#review83483 -----------------------------------------------------------
Ship it! Looks good to me. - Dennis Nienhüser On Aug. 5, 2015, 4:17 p.m., Gábor Péterffy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124622/ > ----------------------------------------------------------- > > (Updated Aug. 5, 2015, 4:17 p.m.) > > > Review request for Marble. > > > Repository: marble > > > Description > ------- > > This patch contains a minimalistic ui for routing. > > - Planning route to destination from current position > - Planning route from setted departure > - Planning route with waypoints > - Instructions in the menu > > What is missing: > - Waypoint markers for the map, these will come in a new patch. > - Maybe a loading screen while route planning in progress > > > Diffs > ----- > > src/lib/marble/declarative/Routing.h 182fe64 > src/lib/marble/declarative/Coordinate.cpp bbdc829 > src/lib/marble/declarative/MarbleDeclarativePlugin.cpp 3bf2198 > src/lib/marble/declarative/MarbleQuickItem.h cefb0c0 > src/apps/marble-maps/RoutePlanViewer.qml PRE-CREATION > src/apps/marble-maps/NavigationSetup.qml PRE-CREATION > src/apps/marble-maps/MainScreen.qml b8dc7b0 > data/android/drawable-xxxhdpi/walk.png PRE-CREATION > data/android/drawable-xxxhdpi/navigation.png PRE-CREATION > data/android/drawable-xxxhdpi/car.png PRE-CREATION > src/lib/marble/declarative/SearchBackend.cpp fdef702 > src/lib/marble/declarative/SearchBackend.h 4981c31 > src/lib/marble/declarative/Routing.cpp dbbcb80 > src/lib/marble/declarative/MarbleQuickItem.cpp f8c396f > src/apps/marble-maps/Search.qml 9b712c8 > src/lib/marble/declarative/Coordinate.h db2ab9f > src/apps/marble-maps/NavigationSetupButton.qml PRE-CREATION > src/apps/marble-maps/MarbleMaps.qrc 0168893 > data/android/drawable-xxxhdpi/waypoint.png PRE-CREATION > data/android/drawable-xxxhdpi/place.png PRE-CREATION > data/android/drawable-xxxhdpi/map.png PRE-CREATION > data/android/drawable-xxxhdpi/bike.png PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/124622/diff/ > > > Testing > ------- > > It works well on my device, everything fits and has right positioning. > > > File Attachments > ---------------- > > bike.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/f39163fa-12ab-4aa2-a787-9c43e5d54177__bike.png > walk.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/21576ef7-f5c8-4cb2-b731-63e14bdb51db__walk.png > place.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/373c2a86-625c-4380-b699-0acbdd269a9d__place.png > navigation.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/7adf250e-4a25-4ddd-a1a6-5e9e6d905ee1__navigation.png > map.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/efe967e2-0ae1-4ac0-8678-8b2cea4e32b1__map.png > car.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/9e5f5f87-352c-4e70-b7ae-cdf8be296863__car.png > waypoint.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/2fcdc0b5-6552-4a89-bf55-ecf9b50e1605__waypoint.png > Screenshot I. > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/08bed68d-5ec3-43a1-98ac-467fc8aa94da__Screenshot_2015-08-04-22-39-57.png > Screenshot II. > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/f2cc5de7-f966-496a-bbe1-2b24859511c1__Screenshot_2015-08-04-22-39-52.png > Screenshot III > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/76795bf1-590f-4e61-bb67-1cb10ed6870b__Screenshot_2015-08-04-22-39-24.png > Screenshot IV > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/8f8d1795-d751-4ee9-bec0-eee3404b15b4__Screenshot_2015-08-04-22-39-11.png > Screenshot V > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/04/ba50137d-21ec-4759-b6f4-38cbe882f47e__Screenshot_2015-08-04-22-38-03.png > > > Thanks, > > Gábor Péterffy > >
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
