reach/router

Active link is not asserted for URL with special characters

Open

#320 aberto em 1 de nov. de 2019

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (352 forks)batch import
good first issuehelp wanted

Métricas do repositório

Stars
 (6.917 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

I think that here:

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

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

Guia do colaborador

Direção de pesquisa
Observe a linha vinculada no index.js (L406 L407) e substitua location.pathname por decodeURIComponent(location.pathname) para lidar corretamente com URLs com caracteres especiais.
Pilha de tecnologia
javascriptreact
Domain
frontendweb dev
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85