Feature/fix tree #67
Feature/fix tree #67
Conversation
|
|
||
| private void label4_Click(object sender, EventArgs e) | ||
| { | ||
|
|
There was a problem hiding this comment.
Algum motivo para esse botão não ter ação?
| using StarkBankExcel.Forms; | ||
| using System.Windows.Forms; |
There was a problem hiding this comment.
A posição anterior desse import fazia mais sentido, não?
Eu deixaria na linha 5
| @@ -33,6 +33,7 @@ private void InternalStartup() | |||
| this.button5.Click += new System.EventHandler(this.button5_Click); | |||
| this.Startup += new System.EventHandler(this.Planilha15_Startup); | |||
| this.Shutdown += new System.EventHandler(this.Planilha15_Shutdown); | |||
|
|
|||
There was a problem hiding this comment.
Dei uma olhada em outros arquivos e parece que seguimos o padrão de não ter essa quebra de linha antes de fechar as chaves
| @@ -154,7 +154,7 @@ public sealed partial class GetStatement : Microsoft.Office.Tools.Excel.Workshee | |||
| this.button1.BackColor = System.Drawing.SystemColors.Control; | |||
| this.button1.ForeColor = System.Drawing.SystemColors.ControlText; | |||
| this.button1.Name = "button1"; | |||
| this.button1.Text = "Baixar Extrato"; | |||
| this.button1.Text = "Extrato"; | |||
There was a problem hiding this comment.
Esse botão faz o GET do extrato ou faz o download do extrato para a maquina do cliente?
Se a função for download, acredito que faça sentido manter o texto "baixar"
| using Newtonsoft.Json.Linq; | ||
| using StarkBankExcel.Forms; | ||
| using System.Windows.Forms; | ||
| using StarkBankExcel.Resources; | ||
| using System.Collections.Generic; |
There was a problem hiding this comment.
Faz sentido deixar os imports de system todos juntos no começo do arquivo?
Ex:
using System.IO;
using System.Data;
using System.Drawing;
using System.Windows.Forms;
using System.Collections.Generic;
| @@ -98,6 +98,7 @@ private void Help_Click(object sender, EventArgs e) | |||
| { | |||
| ViewHelpForm viewHelpForm = new ViewHelpForm(); | |||
| viewHelpForm.ShowDialog(); | |||
|
|
|||
There was a problem hiding this comment.
Podemos tirar esse espaço antes de fechar a chave?
| @@ -110,6 +111,11 @@ private void button11_Click(object sender, EventArgs e) | |||
| Globals.Planilha11.Activate(); | |||
| } | |||
|
|
|||
| private void button3_Click(object sender, EventArgs e) | |||
| { | |||
|
|
|||
There was a problem hiding this comment.
Esse botão sem ação faz sentido?
No description provided.