[Feature] CodeBuity&Refactor #49, [Feature] Code review tool #10

This commit is contained in:
Markus Hofknecht 2020-03-23 12:25:02 +01:00
parent 2b9822c24b
commit a59a38d3b7
7 changed files with 28 additions and 28 deletions

View file

@ -720,7 +720,7 @@ namespace SystemTrayMenu.Controls
FormBorderStyle = FormBorderStyle.Sizable;
TabPanelDetails.Dock = DockStyle.Fill;
tableLayoutPanel1.Dock = DockStyle.Fill;
this.AutoSize = false;
AutoSize = false;
SizeGripStyle = SizeGripStyle.Show;
Size = new Size(580, Size.Height);
MoreRichTextBox.Visible = false;

View file

@ -11,20 +11,20 @@ namespace SystemTrayMenu.Controls
public AskHotKeyForm()
{
InitializeComponent();
this.Text = Language.Translate("Shortcut key");
this.labelCaption.Text = $"{Language.Translate("Shortcut key")} " +
Text = Language.Translate("Shortcut key");
labelCaption.Text = $"{Language.Translate("Shortcut key")} " +
$"{Language.Translate("(e.g. F10)")}";
this.labelText.Text =
labelText.Text =
Language.Translate("CTRL") + " + " +
Language.Translate("ALT") + " + ?";
this.buttonOk.Text = Language.Translate("buttonOk");
buttonOk.Text = Language.Translate("buttonOk");
}
private void ButtonOk_Click(object sender, EventArgs e)
{
this.DialogResult = DialogResult.None;
this.Close();
DialogResult = DialogResult.None;
Close();
}
protected override bool ProcessCmdKey(ref Message msg, Keys keys)
@ -103,14 +103,14 @@ namespace SystemTrayMenu.Controls
case Keys.F23:
case Keys.F24:
NewHotKey = keys.ToString();
this.DialogResult = DialogResult.OK;
this.Close();
DialogResult = DialogResult.OK;
Close();
break;
case Keys.Back:
case Keys.Delete:
NewHotKey = string.Empty;
this.DialogResult = DialogResult.OK;
this.Close();
DialogResult = DialogResult.OK;
Close();
break;
default:
break;

View file

@ -9,14 +9,14 @@ namespace SystemTrayMenu.Controls
string buttonOk)
{
InitializeComponent();
this.Text = hintTitle;
this.labelHint.Text = hintText;
Text = hintTitle;
labelHint.Text = hintText;
this.buttonOk.Text = buttonOk;
}
private void ButtonOk_Click(object sender, EventArgs e)
{
this.Close();
Close();
}
}
}

View file

@ -56,7 +56,7 @@ namespace SystemTrayMenu
DataGridViewCellStyle dgvCellStyle = new DataGridViewCellStyle();
dgvCellStyle.SelectionBackColor = MenuDefines.ColorSelectedItem;
dgvCellStyle.SelectionForeColor = Color.Black;
this.dgv.DefaultCellStyle = dgvCellStyle;
dgv.DefaultCellStyle = dgvCellStyle;
VScrollBar scrollBar = dgv.Controls.OfType<VScrollBar>().First();
scrollBar.MouseWheel += dgv_MouseWheel;
@ -282,12 +282,12 @@ namespace SystemTrayMenu
public void SetTitleColorDeactive()
{
this.labelTitle.ForeColor = Color.LightGray;
labelTitle.ForeColor = Color.LightGray;
}
public void SetTitleColorActive()
{
this.labelTitle.ForeColor = Color.Black;
labelTitle.ForeColor = Color.Black;
}
private void LabelTitle_MouseDoubleClick(object sender, MouseEventArgs e)
@ -300,12 +300,12 @@ namespace SystemTrayMenu
private void LabelTitle_MouseEnter(object sender, EventArgs e)
{
this.labelTitle.BackColor = MenuDefines.ColorTitleSelected;
labelTitle.BackColor = MenuDefines.ColorTitleSelected;
}
private void LabelTitle_MouseLeave(object sender, EventArgs e)
{
this.labelTitle.BackColor = MenuDefines.ColorTitleBackground;
labelTitle.BackColor = MenuDefines.ColorTitleBackground;
}
protected override CreateParams CreateParams

View file

@ -23,7 +23,7 @@ namespace SystemTrayMenu.Helper
public Window()
{
// create the handle for the window.
this.CreateHandle(new CreateParams());
CreateHandle(new CreateParams());
}
/// <summary>
@ -55,7 +55,7 @@ namespace SystemTrayMenu.Helper
public void Dispose()
{
this.DestroyHandle();
DestroyHandle();
}
#endregion

View file

@ -37,7 +37,7 @@ namespace SystemTrayMenu.Helper
/// <summary>Default constructor</summary>
public ShellContextMenu()
{
this.CreateHandle(new CreateParams());
CreateHandle(new CreateParams());
}
#endregion
@ -466,7 +466,7 @@ namespace SystemTrayMenu.Helper
// Release all resources first.
ReleaseAll();
_arrPIDLs = GetPIDLs(files);
this.ShowContextMenu(pointScreen);
ShowContextMenu(pointScreen);
}
/// <summary>
@ -479,7 +479,7 @@ namespace SystemTrayMenu.Helper
// Release all resources first.
ReleaseAll();
_arrPIDLs = GetPIDLs(dirs);
this.ShowContextMenu(pointScreen);
ShowContextMenu(pointScreen);
}
/// <summary>
@ -537,7 +537,7 @@ namespace SystemTrayMenu.Helper
TPM.RETURNCMD,
pointScreen.X,
pointScreen.Y,
this.Handle,
Handle,
IntPtr.Zero);
DestroyMenu(pMenu);
@ -1539,7 +1539,7 @@ namespace SystemTrayMenu.Helper
public LocalWindowsHook(HookType hook)
{
m_hookType = hook;
m_filterFunc = new HookProc(this.CoreHookProc);
m_filterFunc = new HookProc(CoreHookProc);
}
public LocalWindowsHook(HookType hook, HookProc func)
{

View file

@ -77,8 +77,8 @@ namespace SystemTrayMenu
data.cbSize = (uint)Marshal.SizeOf(typeof(APPBARDATA));
result = SHAppBarMessage(ABM.GetState, ref data);
int state = result.ToInt32();
this.AlwaysOnTop = (state & ABS.AlwaysOnTop) == ABS.AlwaysOnTop;
this.AutoHide = (state & ABS.Autohide) == ABS.Autohide;
AlwaysOnTop = (state & ABS.AlwaysOnTop) == ABS.AlwaysOnTop;
AutoHide = (state & ABS.Autohide) == ABS.Autohide;
}
}
}