Merge lp:~mxsscott/widelands/1096786 into lp:widelands

Proposed by Mark Scott
Status: Merged
Merged at revision: 6485
Proposed branch: lp:~mxsscott/widelands/1096786
Merge into: lp:widelands
Diff against target: 54 lines (+20/-18)
1 file modified
src/wui/interactive_base.cc (+20/-18)
To merge this branch: bzr merge lp:~mxsscott/widelands/1096786
Reviewer Review Type Date Requested Status
SirVer Approve
Review via email: mp+142182@code.launchpad.net

Description of the change

Indicate direction of steepness of proposed road building - uphill or downhill.

See commit message for a detailed change description.

Adds two new images to the pics/ folder, which are direct horizontal flips of the existing road steepness indicators.

To post a comment you must log in.
Revision history for this message
SirVer (sirver) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added file 'pics/roadb_reddown.png'
0Binary files pics/roadb_reddown.png 1970-01-01 00:00:00 +0000 and pics/roadb_reddown.png 2013-01-07 19:43:25 +0000 differ0Binary files pics/roadb_reddown.png 1970-01-01 00:00:00 +0000 and pics/roadb_reddown.png 2013-01-07 19:43:25 +0000 differ
=== added file 'pics/roadb_yellowdown.png'
1Binary files pics/roadb_yellowdown.png 1970-01-01 00:00:00 +0000 and pics/roadb_yellowdown.png 2013-01-07 19:43:25 +0000 differ1Binary files pics/roadb_yellowdown.png 1970-01-01 00:00:00 +0000 and pics/roadb_yellowdown.png 2013-01-07 19:43:25 +0000 differ
=== modified file 'src/wui/interactive_base.cc'
--- src/wui/interactive_base.cc 2012-12-16 14:29:46 +0000
+++ src/wui/interactive_base.cc 2013-01-07 19:43:25 +0000
@@ -810,26 +810,28 @@
810 if (m_buildroad->get_index(neighb) >= 0)810 if (m_buildroad->get_index(neighb) >= 0)
811 continue; // the road can't cross itself811 continue; // the road can't cross itself
812812
813 int32_t const slope =813 int32_t slope;
814 abs(endpos.field->get_height() - neighb.field->get_height());814
815 int32_t icon;815 if
816816 (Widelands::WALK_E == dir
817 if (slope < 2)817 || Widelands::WALK_NE == dir
818 icon = 1;818 || Widelands::WALK_SE == dir)
819 slope = neighb.field->get_height() - endpos.field->get_height();
820 else
821 slope = endpos.field->get_height() - neighb.field->get_height();
822
823 const char * name = 0;
824
825 if (slope <= -4)
826 name = "pics/roadb_reddown.png";
827 else if (slope <= -2)
828 name = "pics/roadb_yellowdown.png";
829 else if (slope < 2)
830 name = "pics/roadb_green.png";
819 else if (slope < 4)831 else if (slope < 4)
820 icon = 2;832 name = "pics/roadb_yellow.png";
821 else833 else
822 icon = 3;834 name = "pics/roadb_red.png";
823
824 char const * name;
825 switch (icon) {
826 case 1: name = "pics/roadb_green.png"; break;
827 case 2: name = "pics/roadb_yellow.png"; break;
828 case 3: name = "pics/roadb_red.png"; break;
829 default:
830 assert(false);
831 break;
832 }
833835
834 egbase().map().overlay_manager().register_overlay836 egbase().map().overlay_manager().register_overlay
835 (neighb,837 (neighb,

Subscribers

People subscribed via source and target branches

to status/vote changes: