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(header): MenuBurgerContent was sliding down too much and was miss… #221

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

hudson-viana
Copy link
Contributor

…ing the necessary DropdownMenu.Portal tag

Esse PR resolve a task #203. Porém é um workaround pois tudo indica que é um bug do Radix e não da nossa aplicação.

O que é o bug?

Ao clicar no menu hamburguer no mobile (e tablet) o menu desliza demais expondo uma margem superior desnecessária. Após aprofundar no assunto foi constatado também que não só ele expoe margem superior como também fica variando a altura que o menu expande. Conforme vídeo abaixo:

before-bug.mp4

Workaround

Eu tentei várias estratégias e alternativas (listadas abaixo) para solucionar esse bug mas todas elas não tiveram sucesso em solucionar o bug. Então o workaround atual é subir o menu utilizando o sideOffset={} (do Radix) para que mesmo ele variando a distancia que o menu aparece do trigger ele não exponha essa margem superior estranha pois tudo acontece em cima do header e por isso não expoem mais a margem estranha. Vídeo abaixo mostra o workaround.

after-workaround.mp4

Lista das estratégias e alternativas testadas que não solucionaram o bug

@hudson-viana hudson-viana requested a review from davisbento July 18, 2024 20:19
@hudson-viana hudson-viana self-assigned this Jul 18, 2024
Copy link

vercel bot commented Jul 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mentores-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2024 8:21pm

@davisbento davisbento merged commit a3f18c7 into SouJunior:main Jul 18, 2024
1 check passed
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