Skip to content

Commit 4e41d26

Browse files
author
SlavaRa
committed
Add RefactorMenu.MoveMenuItem and small cleanup....
1 parent 102097e commit 4e41d26

File tree

1 file changed

+32
-64
lines changed

1 file changed

+32
-64
lines changed
Lines changed: 32 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -1,119 +1,87 @@
11
using System;
2-
using System.Text;
3-
using System.Drawing;
42
using System.Windows.Forms;
5-
using PluginCore.Localization;
63
using PluginCore;
4+
using PluginCore.Localization;
75

86
namespace CodeRefactor.Controls
97
{
108
public class RefactorMenu : ToolStripMenuItem
119
{
12-
private SurroundMenu surroundMenu;
13-
private ToolStripMenuItem batchMenuItem;
14-
private ToolStripMenuItem renameMenuItem;
15-
private ToolStripMenuItem truncateMenuItem;
16-
private ToolStripMenuItem organizeMenuItem;
17-
private ToolStripMenuItem delegateMenuItem;
18-
private ToolStripMenuItem generatorMenuItem;
19-
private ToolStripMenuItem extractMethodMenuItem;
20-
private ToolStripMenuItem extractLocalVariableMenuItem;
21-
2210
public RefactorMenu(Boolean createSurroundMenu)
2311
{
2412
this.Text = TextHelper.GetString("Label.Refactor");
25-
this.renameMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.Rename")) as ToolStripMenuItem;
26-
this.renameMenuItem.Image = PluginBase.MainForm.FindImage("331");
27-
this.extractMethodMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.ExtractMethod"), null) as ToolStripMenuItem;
28-
this.extractLocalVariableMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.ExtractLocalVariable"), null) as ToolStripMenuItem;
29-
this.delegateMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.DelegateMethods"), null) as ToolStripMenuItem;
13+
this.RenameMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.Rename")) as ToolStripMenuItem;
14+
this.RenameMenuItem.Image = PluginBase.MainForm.FindImage("331");
15+
#region TODO: localize me
16+
MoveMenuItem = DropDownItems.Add("Move...") as ToolStripMenuItem;
17+
//MoveMenuItem = DropDownItems.Add(TextHelper.GetString("Label.Move")) as ToolStripMenuItem;
18+
#endregion
19+
this.ExtractMethodMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.ExtractMethod"), null) as ToolStripMenuItem;
20+
this.ExtractLocalVariableMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.ExtractLocalVariable"), null) as ToolStripMenuItem;
21+
this.DelegateMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.DelegateMethods"), null) as ToolStripMenuItem;
3022
if (createSurroundMenu)
3123
{
32-
this.surroundMenu = new SurroundMenu();
33-
this.DropDownItems.Add(this.surroundMenu);
24+
this.SurroundMenu = new SurroundMenu();
25+
this.DropDownItems.Add(this.SurroundMenu);
3426
}
3527
this.DropDownItems.Add(new ToolStripSeparator());
36-
this.generatorMenuItem = new ToolStripMenuItem(TextHelper.GetString("Label.InvokeCodeGenerator"), null, null, createSurroundMenu ? Keys.Control | Keys.Shift | Keys.D1 : Keys.None);
37-
this.DropDownItems.Add(this.generatorMenuItem);
28+
this.CodeGeneratorMenuItem = new ToolStripMenuItem(TextHelper.GetString("Label.InvokeCodeGenerator"), null, null, createSurroundMenu ? Keys.Control | Keys.Shift | Keys.D1 : Keys.None);
29+
this.DropDownItems.Add(this.CodeGeneratorMenuItem);
3830
this.DropDownItems.Add(new ToolStripSeparator());
39-
this.organizeMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.OrganizeImports"), null) as ToolStripMenuItem;
40-
this.truncateMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.TruncateImports"), null) as ToolStripMenuItem;
31+
this.OrganizeMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.OrganizeImports"), null) as ToolStripMenuItem;
32+
this.TruncateMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.TruncateImports"), null) as ToolStripMenuItem;
4133
this.DropDownItems.Add(new ToolStripSeparator());
42-
this.batchMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.BatchProcess"), null) as ToolStripMenuItem;
34+
this.BatchMenuItem = this.DropDownItems.Add(TextHelper.GetString("Label.BatchProcess"), null) as ToolStripMenuItem;
4335
}
4436

4537
/// <summary>
4638
/// Accessor to the SurroundMenu
4739
/// </summary>
48-
public SurroundMenu SurroundMenu
49-
{
50-
get { return this.surroundMenu; }
51-
}
40+
public SurroundMenu SurroundMenu { get; private set; }
5241

5342
/// <summary>
5443
/// Accessor to the BatchMenuItem
5544
/// </summary>
56-
public ToolStripMenuItem BatchMenuItem
57-
{
58-
get { return this.batchMenuItem; }
59-
}
45+
public ToolStripMenuItem BatchMenuItem { get; private set; }
6046

6147
/// <summary>
6248
/// Accessor to the RenameMenuItem
6349
/// </summary>
64-
public ToolStripMenuItem RenameMenuItem
65-
{
66-
get { return this.renameMenuItem; }
67-
}
50+
public ToolStripMenuItem RenameMenuItem { get; private set; }
51+
52+
/// <summary>
53+
/// Accessor to the MoveMenuItem
54+
/// </summary>
55+
public ToolStripMenuItem MoveMenuItem { get; private set; }
6856

6957
/// <summary>
7058
/// Accessor to the TruncateMenuItem
7159
/// </summary>
72-
public ToolStripMenuItem TruncateMenuItem
73-
{
74-
get { return this.truncateMenuItem; }
75-
}
60+
public ToolStripMenuItem TruncateMenuItem { get; private set; }
7661

7762
/// <summary>
7863
/// Accessor to the OrganizeMenuItem
7964
/// </summary>
80-
public ToolStripMenuItem OrganizeMenuItem
81-
{
82-
get { return this.organizeMenuItem; }
83-
}
65+
public ToolStripMenuItem OrganizeMenuItem { get; private set; }
8466

8567
/// <summary>
8668
/// Accessor to the ExtractMethodMenuItem
8769
/// </summary>
88-
public ToolStripMenuItem ExtractMethodMenuItem
89-
{
90-
get { return this.extractMethodMenuItem; }
91-
}
70+
public ToolStripMenuItem ExtractMethodMenuItem { get; private set; }
9271

9372
/// <summary>
9473
/// Accessor to the DelegateMenuItem
9574
/// </summary>
96-
public ToolStripMenuItem DelegateMenuItem
97-
{
98-
get { return this.delegateMenuItem; }
99-
}
75+
public ToolStripMenuItem DelegateMenuItem { get; private set; }
10076

10177
/// <summary>
10278
/// Accessor to the ExtractLocalVariableMenuItem
10379
/// </summary>
104-
public ToolStripMenuItem ExtractLocalVariableMenuItem
105-
{
106-
get { return this.extractLocalVariableMenuItem; }
107-
}
80+
public ToolStripMenuItem ExtractLocalVariableMenuItem { get; private set; }
10881

10982
/// <summary>
11083
/// Accessor to the CodeGeneratorMenuItem
11184
/// </summary>
112-
public ToolStripMenuItem CodeGeneratorMenuItem
113-
{
114-
get { return this.generatorMenuItem; }
115-
}
116-
85+
public ToolStripMenuItem CodeGeneratorMenuItem { get; private set; }
11786
}
118-
119-
}
87+
}

0 commit comments

Comments
 (0)