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

Commit Hook null at kde.org
Thu May 30 14:10:26 UTC 2013


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

(Updated May 30, 2013, 2:10 p.m.)


Status
------

This change has been marked as submitted.


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/865255c5/attachment.html>


More information about the Marble-devel mailing list