mateusmangueira / dev-web-project

Repositório para Projeto da disciplina de Desenvolvimento Web - UFCG 2021.1
0 stars 0 forks source link

Estrutura confusa na página de registro #5

Closed thayannevls closed 2 years ago

thayannevls commented 2 years ago

Acredito que haja um erro na estrutura da página nessa parte. Não faz muito sentido o modal está como children do botão, acredito que se estivesse funcionando o onClick do botão, faria o modal renderizar estranho na página.

O ideal seria talvez isolar o modal de registro em um novo componente já que imagino que ele vá ter mais lógica no futuro, e daí seria algo assim:

return (
    <div>
      <Link to="/">Login</Link>
      <hr></hr>
      <CustomButton onClick={handleModal} text="Registrar"/>
        <RegisterModal show={showModal} handleClose={handleModal} />
    </div>
  )
mateusmangueira commented 2 years ago

Faz sentindo, retirei o Modal por enquanto do projeto... Caso, futuramente, eu precise eu retorno e sigo a estrutura que você me passou como feedback.

Obrigado!