Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Desafio front-end size bay #46

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

Desafio front-end size bay #46

wants to merge 25 commits into from

Conversation

phap9060
Copy link

@phap9060 phap9060 commented Mar 7, 2024

Rodar a aplicação

  • Como requisitado pelas regras do desafio, será necessário apenas um comando para rodar a aplicação mas como a aplicação está em uma pasta dentro do repositório é necessário seguir os seguintes passos :
  1. cd Did-I ( Para entrar na pasta da aplicação )
  2. npm i ( Para isntalar as dependências e após isso rodar a aplicação )

Pensamentos durante o desafio

  • Primeiro a forma como componentizei, como indicado nas regras do desafio eu optei por uma componentização simples. Eu tive algumas dúvidas de como faria para componentizar os inputs, afinal apesar de poder ter apenas um components e através de propriedades fazer com que ele se adequasse a necessidade, eu achei que seria melhor ter eles em componentes separados pois apesar de serem inputs considerei que o design completamente diferente fosse melhor ter essa regras separadas para maior manutenabilidade.
  • Os components possuem 3 arquivos que são de estilização, tipos e o componente em si, mas caso haja uma constant que seja especificamente utilizada para um component, tal qual utilizei no componente de Data nessa aplicação, mantive dentro da pasta do component, haja visto que é de uso muito particular.
  • Para animação de quando as tasks são totalmente concluídas optei por utilizar Lotties por ser uma animação leve, fácil de manusear e de ter dentro da aplicação.

@vaporwavie
Copy link
Contributor

vaporwavie commented Apr 2, 2024

Hi @phap9060 thanks for your submission! We're currently high-demanded on a project and for now we don't have much spare time to do the code reviews, sadly. As soon as this project goes by, we'll make sure to have your code reviewed :)

PS: our frontend developer position was already filled, so this will be CR-intended only.

@vaporwavie vaporwavie added the test-only PRs that were opened by Developers who want to get CRed by Sizebay. label Apr 2, 2024
@vaporwavie vaporwavie requested a review from notlesz April 2, 2024 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test-only PRs that were opened by Developers who want to get CRed by Sizebay.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants