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

[Bug Fix] Girar a tela do celular (vertical <-> horizontal) logo após publicar uma resposta gera mensagem de erro #515

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

aprendendofelipe
Copy link
Collaborator

Buscando entender comportamentos que ocorrem de maneiras diferentes no computador e no celular, encontrei mais esse bug.

Pelo que vi nos deploys antigos, o bug já está presente há bastante tempo, mas como ocorre em uma situação bem específica, não deve ser considerado urgente. Se bem que a solução é bem simples.

Como ocorre o problema

Ao clicar em responder, o componente Content que inicialmente estava renderizado em CompactMode passa para EditMode.

Inicialmente o contentObject só tem o id do conteúdo pai.

Após submeter a resposta, o contentObject é atualizado com os dados da resposta e o componente muda para ViewMode.

Acontece que ao mudar a disposição da tela, quase tudo é renderizado novamente e alguns estados são mantidos, mas outros são perdidos.

O componente recebe novamente o content, o que acaba zerando o contentObject, ficando novamente apenas com o id do conteúdo pai, mas continua renderizado em ViewMode, o que causa o erro.

Solução proposta

Usar a sintaxe de espalhamento no setContentObject para não perder os dados atuais no contentObject e apenas atualizar o que vier diferente no content.

@vercel
Copy link

vercel bot commented Jul 17, 2022

@aprendendofelipe is attempting to deploy a commit to the TabNews Team on Vercel.

To accomplish this, @aprendendofelipe needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

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.

2 participants