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

Bug de ícone 100% escuro ao adicionar TabNews no celular #587

Closed
filipedeschamps opened this issue Jul 27, 2022 · 6 comments
Closed

Bug de ícone 100% escuro ao adicionar TabNews no celular #587

filipedeschamps opened this issue Jul 27, 2022 · 6 comments
Labels
front Envolve modificações no frontend

Comments

@filipedeschamps
Copy link
Owner

filipedeschamps commented Jul 27, 2022

Contexto

Ao testar o PR #553 notei que o ícone do TabNews está 100% escuro no celular. Isto não está relacionado ao PR #553 pois conseguir reproduzir em Produção antes do merge, este foi o teste:

IMG_3796

Execução

Entender porque antes funcionava (aparecia o ícone com o fundo branco), se tem relação com o tema Dark do celular, ou se é o fato de estar sendo mostrado um ícone com traçado escuro contra um fundo "transparente".

@filipedeschamps filipedeschamps added the front Envolve modificações no frontend label Jul 27, 2022
@imattferreira
Copy link

imattferreira commented Jul 28, 2022

@filipedeschamps testei aqui no meu phone (um Xiaomi com Android 10 e utilizando um force dark mode nos apps), e tive o msm problema só q com duas diferenças:

  • quando fui add a home no modal do Chrome o ícone do TabNews apareceu corretamente;
  • dps de add na home o ícone ficou totalmente branco;

Vou deixar aqui um vídeo mostrando esse comportamento:
https://user-images.githubusercontent.com/52766091/181403839-a21a69cc-02a0-4ede-b931-d09c760a2157.mp4

@filipedeschamps
Copy link
Owner Author

Que comportamento interessantíssimo!

Será que tem como definir um ícone fixo para quando é mobile?

@filipedeschamps
Copy link
Owner Author

Fechado por #595

@filipedeschamps
Copy link
Owner Author

Aliás, @MattFerreira18 se puder verificar se o comportamento consertou para você seria show 🤝

@imattferreira
Copy link

@filipedeschamps @rodrigoKulb Show, funcionou certinho para mim

@filipedeschamps
Copy link
Owner Author

Para deixar registrado aqui o antes e depois da implementação do @rodrigoKulb

Antes

IMG_3798

Depois

Nota que o manifest até altera o título para algo melhor:

IMG_3800

Home screen

IMG_3802

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front Envolve modificações no frontend
Projects
None yet
Development

No branches or pull requests

2 participants