frontend icon indicating copy to clipboard operation
frontend copied to clipboard

Published 20 hours ago • verified publisher icon SOS-RS

[ShelterListItem] Substitui onClick e useNavigate por <Link> para melhorar acessibilidade

Open sombraSoft opened this issue 9 months ago • 2 comments

Esse PR remove a lógica de navegação para favorecer o uso de links html ao invés da navegação imperativa via javascript. Dessa maneira fica mais claro que todo o card é clicável, além de melhorar a acessibilidade e possibilidade de abrir a página em nova aba. Foi também adicionado hover:bg-accent no card para melhorar a experiência. chrome_9Gr7qKs9uN

sombraSoft avatar May 10 '24 16:05 sombraSoft

Quando puder resolve o conflito que pretendo subir seu PR. Ficou muito bom.

fagundesjg avatar May 11 '24 07:05 fagundesjg

@fagundesjg está pronto, resolvi os conflitos. Para substituir o onClick agora tive que me livrar da prop que era passada onSelectShelter. Tudo o que ela fazia era chamar um navigate.

sombraSoft avatar May 11 '24 15:05 sombraSoft

foi feito um PR novo com a mesma funcionalidade pra contornar os problemas de conflito nessa branch https://github.com/SOS-RS/frontend/pull/128

sombraSoft avatar May 13 '24 19:05 sombraSoft