dotnet/aspnetcore

Suggestion: Expose a parameter of `ShouldMatch` for `<NavLink />`

Fechada

#18.163 aberto em 7 de jan. de 2020

Ver no GitHubGuia do colaborador
 (6 comentários) (1 reação) (0 responsável)C# (10.653 forks)batch import
Priority:1affected-fewarea-blazorenhancementfeature-blazor-builtin-componentshelp wantedseverity-minor

Métricas do repositório

Stars
 (37.933 estrelas)
Métricas de merge de PR
 (Mesclagem média 16d 9h) (258 fundiu PRs em 30d)

Description

Is your feature request related to a problem? Please describe.

I want to custom the matching behavior of <NavLink />.

Let's say we want to process a url with a hash(See this thread on SO) :

http://localhost/#an-element-id

Sometimes we might think this url matches http://localhost , while some other users don't think so.

It would be great if we are allowed to custom this matching behavior. Usually, we can custom by inheritance or composition. However, the matching logic is hardcoded for the present.

  1. There's no public/protected ShouldMatch(string) method exposed. So it's difficult to achieve it by inheritance
  2. There's no ShouldMatch parameter exposed. It would be great if we can pass a ShouldMatch [Parameter] to this component.

Describe the solution you'd like

Expose a [Parameter] of ShouldMatch(string):

  • if user provides such a parameter, use it.
  • Otherwise, use the default logic.

Guia do colaborador

Direção de pesquisa
Comece examinando o componente NavLink em src/Components/Web/src/Routing/NavLink.cs, focando no método ShouldMatch embutido. Procure por padrões de parâmetros semelhantes na base de código. Implemente um novo parâmetro ShouldMatch do tipo Func<string, bool>? que substitui o comportamento de correspondência padrão. Certifique se de que o parâmetro seja opcional e documente o. Também verifique os testes existentes para NavLink e adicione um caso de teste para correspondência personalizada. Revise quaisquer PRs ou comentários vinculados para obter mais orientações.
Pilha de tecnologia
csharp
Domain
frontend
Tipo Issue
Funcionalidade
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#BlazorGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70