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

feat: adiciona notificacao mobile #312

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

kaiocp
Copy link

@kaiocp kaiocp commented Oct 19, 2024

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 #217

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

antes depois
Screenshot 2024-10-19 at 15 47 14 Screenshot 2024-10-19 at 15 46 47
preview no celular
ScreenRecording_10-19-2024.15-44-04_1.MP4

Documentação

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

Copy link

netlify bot commented Oct 19, 2024

Deploy Preview for querido-diario-plataforma ready!

Name Link
🔨 Latest commit 4eab554
🔍 Latest deploy log https://app.netlify.com/sites/querido-diario-plataforma/deploys/671e90c634941e0008c9499a
😎 Deploy Preview https://deploy-preview-312--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 site configuration.

@kaiocp kaiocp marked this pull request as ready for review October 19, 2024 18:48
@lguima
Copy link
Collaborator

lguima commented Oct 22, 2024

fala @kaiocp, valeu pela contribuição!

ano passado tinha rolado outra PR tb: #225

eu reparei na sua solução apenas que o tamanho da notificação está extrapolando um pouco a tela do lado direito

Screenshot from 2024-10-22 10-05-36

@kaiocp
Copy link
Author

kaiocp commented Oct 27, 2024

falaaa @lguima, bão?

ano passado tinha rolado outra PR tb: #225

boa!! o @ogecece tinha trocado uma ideia sync comigo e @MoisesMsa sobre o pessoal da trilha de mantenedoras frontend dar continuidade a algum dos prs já abertos e que não haviam sido finalizados ainda, daí lá na issue #217 eu comentei que iria dar prosseguimento ao pr #225. mas continuar na branch tava dando um super trampo, então achei mais prático abrir uma branch nova e, se for mergeado, fechar o outro pr. o que cê acha?

o tamanho da notificação está extrapolando um pouco a tela do lado direito

hmmmm, acho que não tinha catado isso porque no breakpoint mobile que eu tava testando (425px) não rolava esse overflow, mas acredito que com o patch do 4eab554 isso tenha se resolvido. muito obrigado por ter osbervado!! cê consegue validar pra mim se aí tá 💯 também, por favor?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adicionar botão de notificações no modo mobile
3 participants