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

Torsten Rahn tackat at kde.org
Wed Aug 5 13:36:51 UTC 2015


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



src/apps/marble-maps/MainScreen.qml (line 181)
<https://git.reviewboard.kde.org/r/124622/#comment57686>

    There is no word such as "Setted"
    Make that 
    
    departureIsSet()
    destinationIsSet()
    
    or (...isSpecified())
    or
    
    departureAvailable()
    destinationAvailable()



src/apps/marble-maps/Search.qml (line 13)
<https://git.reviewboard.kde.org/r/124622/#comment57685>

    Signals are always named in past/present perfect tense. This should be named:
    
    signal navigationRequested()


- Torsten Rahn


On Aug. 5, 2015, 11:16 vorm., 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, 11:16 vorm.)
> 
> 
> 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
> -----
> 
>   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/e8f559ad/attachment.html>


More information about the Marble-devel mailing list