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.

Sign up for GitHub

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

Jump to bottom

Ajuste alinhamento das notificações com ícone #307

Open
MoisesMsa opened this issue Oct 12, 2024 · 2 comments
Open

Ajuste alinhamento das notificações com ícone #307

MoisesMsa opened this issue Oct 12, 2024 · 2 comments
Assignees
@devcaiosantos
Labels
bug Something isn't working

Comments

@MoisesMsa
Copy link
Collaborator

Passos para reproduzir

  1. Abrir as notificações em diferentes resoluções
  2. Abrir as notificações em diferentes navegadores (que não seja o chrome)

Comportamento atual

O triângulo que aponta para o ícone está mais a direita

Comportamento esperado

O modal de notificações deve estar centralizado com o ícone do sino em qualquer resolução ou navegador

Informações adicionais

Nesse comentário há mais detalhes sobre o comportamento dos valores de width e margin que são calculados dinamicamente para o container das notificações
#305 (comment)
@MoisesMsa MoisesMsa added the bug Something isn't working label Oct 12, 2024
@devcaiosantos
Copy link
Contributor

Olá, se não tiver ninguém responsável ainda, gostaria de trabalhar com essa issue.

@ogecece @maluojuara

@devcaiosantos devcaiosantos mentioned this issue Nov 25, 2024
Merged
10 tasks
@devcaiosantos
Copy link
Contributor

Criei um PR! #323

@ogecece

Fico a disposição

MoisesMsa added a commit that referenced this issue Dec 12, 2024
@MoisesMsa
fix: Posicionamento dialog notificacao (#323)
957acc6 
Usando como referência o "botão"

**Português (BR)** | [English
(US)](?quick_pull=1&template=PULL_REQUEST-en-US.md)

## Comunidade

* [x] Eu li e segui o [Guia de
Contribuição](https://github.com/okfn-brasil/querido-diario-frontend/blob/main/docs/CONTRIBUTING.md).
* [x] Eu li e segui o [Código de
Conduta](https://docs.queridodiario.ok.org.br/pt-br/latest/contribuindo/codigo-de-conduta.html).

## Tipo de alteração

* [x] 🐞 Correção de problema
* [ ] ✨ Melhoria ou nova funcionalidade
* [ ] 📰 Nova postagem no blog

## Issues relacionadas
#307 

## Validação

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

## Evidências

Antes:

![image](https://github.com/user-attachments/assets/f8bb0d07-9745-4588-8f51-481a20dc2000)

Depois:

https://github.com/user-attachments/assets/3d2bd9e0-e616-438b-b7ea-ea62d9d2b9b7

## Documentação

* [ ] A documentação deste repositório foi atualizada (quando
necessário).
* [ ] Esta alteração requer que a documentação externa seja atualizada.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devcaiosantos devcaiosantos

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MoisesMsa @devcaiosantos