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

fix: collapse main composer #615

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

joaomarcelo09
Copy link

@joaomarcelo09 joaomarcelo09 commented Sep 7, 2024

Closes #541

Feature

N/A

Bugfix
  • Description
    Quando o composer principal colapsa, o texto é perdido.

  • Cause
    A validação era que, se props.isOpen for verdadeiro, ele retorna o conteúdo, mas se não for, retorna nulo. Assim, o componente não estava sendo escondido; ele estava sendo desmontado.

  • Solution
    Eu alterei a validação para que, se props.isOpen for verdadeiro, envie a variante 'expanded' para a prop animate no motion, e se for falso, envie 'collapsed'.

Changelog N/A
Visual evidences 🖼️
Checklist
  • Issue linked
  • Build working correctly
  • Tests created
Additional info N/A

Copy link
Contributor

@JpBurgarelli JpBurgarelli left a comment

Choose a reason for hiding this comment

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

Opa, João, tudo bem? Seu PR tem algumas falhas que você precisa verificar nos arquivos de teste Accordion.spec.tsx, sideBar.spec.tsx, e SocialAccordion.spec.tsx. Dentro da sua branch, antes de commitar as coisas, rode um pnpm test.

No template do PR, você precisa linkar o seu PR com a task que você pegou. Por exemplo, se o ID da task que você pegou é 642, logo após o 'close', coloque close #642. Por padrão, a descrição do PR aqui é em português, apesar de o código ter que ser em inglês.

Copy link
Contributor

@DominMFD DominMFD left a comment

Choose a reason for hiding this comment

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

Arruma os testes e atualiza a branch

@Luiginicoletti
Copy link
Member

Bom dia meu mano, ajusta o CI/CD pra gente dar continuidade ao review, qualquer duvida chama nois.

salonline-youtuber-curly-tamo-tgzsvyb4ezq7b1bwih

src/components/Accordion/Accordion.tsx Outdated Show resolved Hide resolved
src/components/Accordion/Accordion.tsx Outdated Show resolved Hide resolved
src/components/Accordion/Accordion.tsx Outdated Show resolved Hide resolved
@juliaam
Copy link
Contributor

juliaam commented Oct 4, 2024

ajusta os conflitos

@joaomarcelo09 joaomarcelo09 requested a review from juliaam October 7, 2024 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Pending Approval
Development

Successfully merging this pull request may close these issues.

Ajuste no componente de MainCompose
6 participants