[Marble-devel] Review Request 110722: Better icon for the "Remove via point" button

Thomas Jarosch tomj at simonv.com
Thu May 30 10:27:16 UTC 2013


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

Review request for Marble.


Description
-------

Hi,

the black 'X' to remove a via point in the routing manager
is almost not visible on the N900 (black window background).

The original patch used the red "-" icon,
which is visible on the N900 and also more consistent
with the "+" icon to add a via point.

-> Revert to the previous icon.

Thomas


Diffs
-----

  src/lib/routing/RoutingInputWidget.cpp 28db48d 

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


Testing
-------


Thanks,

Thomas Jarosch

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


More information about the Marble-devel mailing list