reach/router

Active link is not asserted for URL with special characters

Open

#320 aperta il 1 nov 2019

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (352 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (6917 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I think that here:

https://github.com/reach/router/blob/28a79e7fc3a3487cb3304210dc3501efb8a50eba/src/index.js#L406-L407

location.pathname should be instead decodeURIComponent(location.pathname)

Guida contributor

Direzione di ricerca
Guarda la riga collegata in index.js (L406 L407) e sostituisci location.pathname con decodeURIComponent(location.pathname) per gestire correttamente gli URL con caratteri speciali.
Tech stack
javascriptreact
Dominio
frontendweb dev
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85