Merge lp:~widelands-dev/widelands/bug_156675_editor_initial_tool into lp:widelands

Proposed by kaputtnik
Status: Merged
Merged at revision: 7961
Proposed branch: lp:~widelands-dev/widelands/bug_156675_editor_initial_tool
Merge into: lp:widelands
Diff against target: 56 lines (+8/-8)
2 files modified
src/editor/editorinteractive.cc (+1/-1)
src/editor/editorinteractive.h (+7/-7)
To merge this branch: bzr merge lp:~widelands-dev/widelands/bug_156675_editor_initial_tool
Reviewer Review Type Date Requested Status
GunChleoc Approve
Review via email: mp+291920@code.launchpad.net

Commit message

Use info tool as initial tool for the editor
Fixes bug 1566675

Description of the change

For the editor use the info tool as initial tool instead of the increase_height tool

Adjusted some spaces to have a nicer look.

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

Continuous integration builds have changed state:

Travis build 1010. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/123111973.
Appveyor build 843. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_156675_editor_initial_tool-843.

Revision history for this message
GunChleoc (gunchleoc) wrote :

LGTM :)

@bunnybot merge

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/editor/editorinteractive.cc'
2--- src/editor/editorinteractive.cc 2016-04-08 15:33:08 +0000
3+++ src/editor/editorinteractive.cc 2016-04-14 16:49:19 +0000
4@@ -686,7 +686,7 @@
5 redo_.set_enabled(false);
6
7 tools_.reset(new Tools());
8- select_tool(tools_->increase_height, EditorTool::First);
9+ select_tool(tools_->info, EditorTool::First);
10 set_sel_radius(0);
11
12 set_need_save(false);
13
14=== modified file 'src/editor/editorinteractive.h'
15--- src/editor/editorinteractive.h 2016-04-06 09:23:04 +0000
16+++ src/editor/editorinteractive.h 2016-04-14 16:49:19 +0000
17@@ -52,7 +52,7 @@
18 struct Tools {
19 Tools()
20 :
21- current_pointer(&increase_height),
22+ current_pointer(&info),
23 use_tool(EditorTool::First),
24 increase_height(decrease_height, set_height),
25 noise_height(set_height),
26@@ -64,9 +64,9 @@
27 {}
28 EditorTool & current() const {return *current_pointer;}
29 using ToolVector = std::vector<EditorTool *>;
30- EditorTool * current_pointer;
31- EditorTool::ToolIndex use_tool;
32- EditorInfoTool info;
33+ EditorTool * current_pointer;
34+ EditorTool::ToolIndex use_tool;
35+ EditorInfoTool info;
36 EditorSetHeightTool set_height;
37 EditorDecreaseHeightTool decrease_height;
38 EditorIncreaseHeightTool increase_height;
39@@ -74,14 +74,14 @@
40 EditorSetTerrainTool set_terrain;
41 EditorDeleteImmovableTool delete_immovable;
42 EditorPlaceImmovableTool place_immovable;
43- EditorSetStartingPosTool set_starting_pos;
44+ EditorSetStartingPosTool set_starting_pos;
45 EditorDeleteBobTool delete_bob;
46 EditorPlaceBobTool place_bob;
47 EditorDecreaseResourcesTool decrease_resources;
48 EditorSetResourcesTool set_resources;
49 EditorIncreaseResourcesTool increase_resources;
50- EditorSetPortSpaceTool set_port_space;
51- EditorUnsetPortSpaceTool unset_port_space;
52+ EditorSetPortSpaceTool set_port_space;
53+ EditorUnsetPortSpaceTool unset_port_space;
54 EditorSetOriginTool set_origin;
55 EditorMakeInfrastructureTool make_infrastructure;
56 };

Subscribers

People subscribed via source and target branches

to status/vote changes: