[Marble-devel] Review Request: Add buttons to save and load routes as *.kml file from within the Routing Widget.

[Florian Eßer]

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

(Updated Sept. 15, 2011, 5:11 p.m.)


Review request for Marble.


Changes
-------

The Load and Save buttons are now located at the very bottom of the widget, below the ListView and the alternative routes ComboBox.

I don't think making them available as RMB actions on the ListView would be very intuitive. The ListView displays the driving directions, while the main feature of the buttons is to save the route request. Also, you can not right-click on the ListView itself if it is full, only on of the items it contains. (As a new user, I would be confused if I could save a route by right-clicking on "Turn left into Whatever Street"...)


Summary
-------

I often use Marble's routing ability to plan somewhat longer routes with a lot
of manually-tweaked waypoints (e.g. for cycling). I would like to be able not
only to export the final gpx/kml file of the route, but also to save and reload
it's "source" (the waypoints).

Basically, this functionality is already built in. The recently edited route is
automatically saved to ~/.local/share/marble/routing/route.kml and restored on
startup. But if I just quickly want to plan another route inbetween, all the
previous work on the "big" route is gone.

Since I don't like to manually backup and restore the route.kml every time, I
created this small patch to add save and load buttons to the RoutingWidget.


This addresses bug 281278.
    http://bugs.kde.org/show_bug.cgi?id=281278


Diffs (updated)
-----

  src/lib/routing/RoutingWidget.cpp 4e84201 
  src/lib/routing/RoutingWidget.ui 7d0620c 

Diff: http://git.reviewboard.kde.org/r/102539/diff


Testing
-------

It worked for me...


Thanks,

Florian

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20110915/529f7d4c/attachment.html>