[Marble-devel] Review Request 124622: Basic routing in Marble Maps

[Gábor Péterffy]

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

(Updated Aug. 5, 2015, 1:16 p.m.)


Review request for Marble.


Changes
-------

Removing undefined references.


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 (updated)
-----

  data/android/drawable-xxxhdpi/bike.png PRE-CREATION 
  data/android/drawable-xxxhdpi/car.png PRE-CREATION 
  data/android/drawable-xxxhdpi/map.png PRE-CREATION 
  data/android/drawable-xxxhdpi/navigation.png PRE-CREATION 
  data/android/drawable-xxxhdpi/place.png PRE-CREATION 
  data/android/drawable-xxxhdpi/walk.png PRE-CREATION 
  data/android/drawable-xxxhdpi/waypoint.png PRE-CREATION 
  src/apps/marble-maps/CircularButton.qml 93a09c0 
  src/apps/marble-maps/MainScreen.qml b8dc7b0 
  src/apps/marble-maps/MarbleMaps.qrc 0168893 
  src/apps/marble-maps/NavigationSetup.qml PRE-CREATION 
  src/apps/marble-maps/NavigationSetupButton.qml PRE-CREATION 
  src/apps/marble-maps/RoutePlanViewer.qml PRE-CREATION 
  src/apps/marble-maps/Search.qml 9b712c8 
  src/lib/marble/declarative/Coordinate.h db2ab9f 
  src/lib/marble/declarative/Coordinate.cpp bbdc829 
  src/lib/marble/declarative/MarbleDeclarativePlugin.cpp 3bf2198 
  src/lib/marble/declarative/MarbleQuickItem.h cefb0c0 
  src/lib/marble/declarative/MarbleQuickItem.cpp f8c396f 
  src/lib/marble/declarative/Routing.h 182fe64 
  src/lib/marble/declarative/Routing.cpp dbbcb80 
  src/lib/marble/declarative/SearchBackend.h 4981c31 
  src/lib/marble/declarative/SearchBackend.cpp fdef702 
  src/lib/marble/routing/Maneuver.cpp c0ab724 

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

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