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

[Thomas Jarosch]

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

(Updated May 30, 2013, 12:29 p.m.)


Review request for Marble.


Changes
-------

As requested by Earthwings: Use the red icon for the n900 version only.
(it's too prominent on the desktop version)


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 (updated)
-----

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