querido-diario-frontend
querido-diario-frontend copied to clipboard
okfn-brasil
Adição notificação mobile
Português (BR) | English (US)
Comunidade
- [x] Eu li e segui o Guia de Contribuição.
- [x] Eu li e segui o Código de Conduta.
Tipo de alteração
- [ ] 🐞 Correção de problema
- [x] ✨ 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.
Deploy Preview for querido-diario-plataforma ready!
| Name | Link |
|---|---|
| Latest commit | 343c86bf833603852a35cfbf9227bb84bf2c8966 |
| 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.
![netlify[bot] avatar](https://avatars.githubusercontent.com/in/13473?v=4 "Published by a pub.dev verified publisher"){kind=small}
Olá! Obrigada pelo retorno, modifiquei aqui no estilo para ajeitar a logo.
@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?
@ogecece obrigada pelo retorno! Entendi, concordo com as suas pontuações, vou propor um novo layout e submeto novamente para review, tudo bem?