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

[BUG] Logo da bóia pós-escrita SOS RS aparece como 🛟 no Desktop #211

Closed
HbLuca opened this issue May 16, 2024 · 9 comments · Fixed by #219 or #304
Closed

[BUG] Logo da bóia pós-escrita SOS RS aparece como 🛟 no Desktop #211

HbLuca opened this issue May 16, 2024 · 9 comments · Fixed by #219 or #304
Assignees
@dadedeandrade
Labels
bug Something isn't working

Comments

@HbLuca
Copy link

HbLuca commented May 16, 2024 •
edited
Loading

Descrição

A bóia circular salva vidas, que na versão mobile aparece de acordo, conforme evidência, no Desktop aparece como um 🛟.

Problema encontrado

🛟 no "sobre nós" e "política de privacidade" após escrita SOS RS

Prioridade

  • Média

Solução proposta

Verificar o logo para adapta-lo ao Desktop

Ambiente

Desktop - Windows 10 (Google Chrome) e Xiaomi Redmi Note 9S

Evidência

desk x mob
@HbLuca HbLuca added the bug Something isn't working label May 16, 2024
@HbLuca
Copy link
Author

HbLuca commented May 16, 2024 •
edited
Loading

Foram testados no Opera GX, Edge e seguem o mesmo comportamento do Google Chrome.
Firefox está de acordo Safari tbm está de acordo.

@rodrigooler rodrigooler self-assigned this May 16, 2024
@dadedeandrade dadedeandrade mentioned this issue May 17, 2024
Merged
@dadedeandrade
Copy link
Contributor

dadedeandrade commented May 17, 2024 •
edited
Loading

Foi criado PR #219 para tratativa deste problema, aparentemente o unicode da boia 1F6DF nao é suportado pela fonte que usamos, para melhor compatibilidade utilizei os icones do Lucide que estão de acordo com os utilizados no figma

@artur-debv
Copy link

Eu Testei no Chrome, agora está ok, pelo menos testei em outros navegadores funcionou

@HbLuca
Copy link
Author

HbLuca commented May 18, 2024

@artur-debv
Aqui segue da mesma forma...
Realizei o teste no Google Chrome, Ópera GX e Edge.
https://www.loom.com/share/294a3d24897349beb48791cf52aaa424?sid=e128f853-113f-4dc9-9782-516d75012b08

@HbLuca HbLuca added this to SOS-RS May 19, 2024
@HbLuca HbLuca moved this to Em desenvolvimento in SOS-RS May 19, 2024
@giggio giggio linked a pull request May 19, 2024 that will close this issue
fix/ add lucide icons to replace emojis so it works across all browsers #219
Merged
@giggio giggio moved this from Em desenvolvimento to Em revisão (dev) in SOS-RS May 19, 2024
@rhuam
Copy link
Contributor

rhuam commented May 20, 2024

Satisfaz as necessidades de produto.

@rodrigooler rodrigooler removed their assignment May 21, 2024
vinny-silveira added a commit that referenced this issue May 22, 2024
@vinny-silveira
fix/ add lucide icons to replace emojis so it works across all browse…
582343d 
…rs (#219)

replaced all emoji occurences with lucide icons to avoid browser
incompatibility #211
@github-project-automation github-project-automation bot moved this from Em revisão (dev) to Entregue (Po + Dev + Infra) in SOS-RS May 22, 2024
@HbLuca
Copy link
Author

HbLuca commented May 22, 2024

Não está de acordo para seguir para merge em produção.
Motivo:
https://dev.sos-rs.com/ está contemplando apenas no sobre nós, falta na parte de política de privacidade, o problema reportado foi em ambos e não apenas em 1.

Vide Loom:
https://www.loom.com/share/2b5cdc8fc88843048975c15ce28595f2?sid=8071a4f4-ceb5-4399-a3b0-ee427395611d

@HbLuca HbLuca reopened this May 22, 2024
@github-project-automation github-project-automation bot moved this from Entregue (Po + Dev + Infra) to Backlog in SOS-RS May 22, 2024
@HbLuca HbLuca moved this from Backlog to Em revisão (dev) in SOS-RS May 22, 2024
@HbLuca HbLuca moved this from Em revisão (dev) to Em desenvolvimento in SOS-RS May 22, 2024
@dadedeandrade
Copy link
Contributor

Feito PR #304 dessa vez foi substituído todas as ocorrências de 🛟 pelo lucide-icons

@rhuam rhuam linked a pull request May 24, 2024 that will close this issue
fix/ replaced all native emoji occurences with lucide icons #304
Merged
@rhuam rhuam moved this from Em desenvolvimento to Em revisão (dev) in SOS-RS May 24, 2024
@larissapissurno
Copy link
Contributor

PR #304 aprovado e mergeado

@larissapissurno larissapissurno moved this from Em revisão (dev) to PR Aprovado e Mergeado in SOS-RS Jun 2, 2024
@larissapissurno larissapissurno moved this from PR Aprovado e Mergeado to Disponível para QA in SOS-RS Jun 18, 2024
@HbLuca HbLuca moved this from Disponível para QA to Em QA in SOS-RS Jun 19, 2024
@HbLuca
Copy link
Author

HbLuca commented Jun 19, 2024

@larissapissurno testado em producao e ja esta contemplado por la, vou encerra-lo.
Vide Loom:
https://www.loom.com/share/1555c63f83a245e4894216fba96f3b66?sid=67318732-48cc-4e4c-9f98-bf6486c266e1

@HbLuca HbLuca moved this from Em QA to Entregue (Po + Dev + Infra) in SOS-RS Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dadedeandrade dadedeandrade

Labels
bug Something isn't working
Projects
SOS-RS
Status: Entregue (Po + Dev + Infra)
Milestone
No milestone
6 participants
@rodrigooler @larissapissurno @rhuam @dadedeandrade @artur-debv @HbLuca