Skip to content

Fix: volta a exibir dialog de notificações#384

Merged
lguima merged 3 commits into
okfn-brasil:mainfrom
alinenaoe:fix-notification-popover
Apr 11, 2026
Merged

Fix: volta a exibir dialog de notificações#384
lguima merged 3 commits into
okfn-brasil:mainfrom
alinenaoe:fix-notification-popover

Conversation

@alinenaoe
Copy link
Copy Markdown
Contributor

@alinenaoe alinenaoe commented Mar 13, 2026

Português (BR) | English (US)

Comunidade

Tipo de alteração

  • 🐞 Correção de problema
  • ✨ Melhoria ou nova funcionalidade
  • 📰 Nova postagem no blog

Issues relacionadas

Resolve #364

Para correção, apenas voltei a referência #notifications da ViewChild no arquivo. Já estava lá antes, mas um PR posterior acabou pegando a versão antiga e removeu por acidente essa parte.

Aproveitei e adicionei autoFocus false na dialog. O padrão é ser true, mas com isso, ao clicar no sininho, a visualização não começa no primeiro item disponível, mas no primeiro link existente, que pode estar bem pra baixo (acho que no vídeo fica mais fácil de ver).

Também commitei o yarn.lock com dependências que foram adicionadas aqui.

Validação

  • Validei a alteração no link gerado pelo bot da Netlify (Deploy Preview/Preview on mobile)
  • Validei o Layout responsivo (desktop/mobile) após a implementação
  • Verifiquei o registro do deploy (Latest deploy log) e nenhum novo alerta ou erro foi adicionado

Evidências

Ao clicar no sininho, as notificações voltam a aparecer.

Aqui é o comportamento sem a inclusão do autoFocus: false:

com_autofocus.mov

E aqui a correção (primeiro eu clico para abrir as notificações e depois dou um tab):

sem_autofocus.mov

Documentação

  • A documentação deste repositório foi atualizada (quando necessário).
  • Esta alteração requer que a documentação externa seja atualizada.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 13, 2026

Deploy Preview for querido-diario-plataforma ready!

Name Link
🔨 Latest commit fea3971
🔍 Latest deploy log https://app.netlify.com/projects/querido-diario-plataforma/deploys/69b3c91aa32d16000863e637
😎 Deploy Preview https://deploy-preview-384--querido-diario-plataforma.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@alinenaoe alinenaoe marked this pull request as ready for review March 13, 2026 08:26
Copy link
Copy Markdown
Collaborator

@lguima lguima left a comment

Choose a reason for hiding this comment

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

Boa, @alinenaoe! Muito obrigado pela contribuição! 🙏

@lguima lguima merged commit f1486f9 into okfn-brasil:main Apr 11, 2026
4 checks 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.

Balão de notificações não aparece mais no site

2 participants