[Marble-devel] Review Request: Bookmarks and current location as routing input

[Dennis Nienhüser]

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

Review request for marble and Torsten Rahn.


Summary
-------

Changes the routing start/destination user interface:
- Use one button instead of two ("remove" button on the right isn't perfect on Maemo)
- Decouple the double-use of the left button ("map input" vs. "center map here")
- Add the possibility to choose a bookmark or the current location as target position

The only drawback I see is that it adds a new menu, i.e. another click and less obvious interface. Another option would be to add a third button with a drop down like this which only has "from map", "bookmark" and "current location" as menu actions. But I'd rather not have three buttons given the rather restricted space we have.
Please review with usability in mind ;-) and make sure to look at the screenshot, it should give a good overview of the changes.


Diffs
-----

  /trunk/KDE/kdeedu/marble/src/lib/routing/RoutingWidget.cpp 1181012 
  /trunk/KDE/kdeedu/marble/src/lib/routing/RoutingInputWidget.cpp 1181012 
  /trunk/KDE/kdeedu/marble/src/lib/routing/RoutingInputWidget.h 1181012 

Diff: http://svn.reviewboard.kde.org/r/5485/diff


Testing
-------

Desktop version only so far, will test on Maemo later.


Screenshots
-----------

Routing Input GUI Changes
  http://svn.reviewboard.kde.org/r/5485/s/516/


Thanks,

Dennis

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/marble-devel/attachments/20100929/06d67480/attachment.htm