Adição notificação mobile

[gabrielliosc]

Português (BR) | English (US)

Comunidade

• Eu li e segui o Guia de Contribuição.
• Eu li e segui o Código de Conduta.

Tipo de alteração

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

Issues relacionadas

Issues que são relacionadas a esta Pull Request.

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

Anexe evidências do antes e do depois da alteração (quando necessário).

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]

✅ Deploy Preview for querido-diario-plataforma ready!

Name	Link
🔨 Latest commit	343c86b
🔍 Latest deploy log	https://app.netlify.com/sites/querido-diario-plataforma/deploys/6540160713a4220008353538
😎 Deploy Preview	https://deploy-preview-225--querido-diario-plataforma.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[lguima]

Oi, @gabrielliosc, valeu por contribuir! 🚀

Reparei que teve um impacto na posição do logo na versão desktop, poderia verificar pra gente, por favor?

Antes

Depois

[gabrielliosc]

Olá! Obrigada pelo retorno, modifiquei aqui no estilo para ajeitar a logo.

[ogecece]

@gabrielliosc desculpe mas como a modificação não estava especificada podemos esse PR pra discutir um pouco a melhor forma de introduzir as notificações em mobile?

Primeiro comentário que é bem mais tranquilo: acho que o menu de notificações deveria colar do lado direito no menu de contexto e não na logo.

Segundo comentário: o modal ocupando apenas parte da tela fica meio esquisito no mobile pois praticamente não dá pra entender nada do que fica atrás. Penso que o melhor seria ocupar a tela inteira, da mesma maneira que o menu de contexto faz. O que acha?

[gabrielliosc]

@ogecece obrigada pelo retorno! Entendi, concordo com as suas pontuações, vou propor um novo layout e submeto novamente para review, tudo bem?