Skip to content

Feature/input with mask - #1

Open
luane-aquino wants to merge 17 commits into
mainfrom
feature/input-with-mask
Open

Feature/input with mask#1
luane-aquino wants to merge 17 commits into
mainfrom
feature/input-with-mask

Conversation

@luane-aquino

Copy link
Copy Markdown
Owner

No description provided.

@guilhermenass guilhermenass left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uma outra coisa: senti muita falta de testes automatizados. Principalmente os testes unitários que nesse caso são indispensáveis.

Se isso aqui fosse um desafio técnico de empresa, com certeza iriam te questionar isso.

Mas o desafio ficou muito bom, consegui rodar normalmente na minha máquina e funcionou. Só se atentar nas validações e nos cenários! Parabéns Lu!

Comment thread src/App.tsx
Comment thread src/App.tsx Outdated
Comment thread src/App.tsx Outdated
@guilhermenass

Copy link
Copy Markdown

Lu, aqui se eu digito um valor pequeno o sufixo está colado.

image

Comment thread src/App.css

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

evitar de usar px E rem. De preferência usar só um dos dois e o REM é mais indicado.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sugestão: quando a gente tem tamanhos em projetos grandes, é legal aplicar um arquivo de tema global para ser usado nas variáveis do css.

inves de:
margin-top: 16px;

usar:
margin-top: var(--VARIAVEL_TAMANHO_16)

como o desafio é simples e não é um projeto grande, não tem problema, mas só pra você saber que é importante padronizar essas coisas. Aqui no projeto que estamos, temos um arquivo para fontes, cores, tamanhos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants