ionic-team/ionic-framework

feat: Improved types for react useIonModal

Open

#28.680 aberto em 11 de dez. de 2023

Ver no GitHub
 (8 comments) (0 reactions) (0 assignees)TypeScript (13.381 forks)batch import
help wantedpackage: reacttype: feature request

Métricas do repositório

Stars
 (52.503 stars)
Métricas de merge de PR
 (Mesclagem média 4d 10h) (56 fundiu PRs em 30d)

Description

Prerequisites

Describe the Feature Request

useIonModal should type props parameter.

Describe the Use Case

useIonModal currently allows any for component props. This is dangerous and has caused many crashes and unintended behaviors in my experience.

Currently:

export declare function useIonModal(component: ReactComponentOrElement, componentProps?: any): UseIonModalResult;

Describe Preferred Solution

export declare function useIonModal<P>(component: React.ComponentClass<P> | React.FC<P>, componentProps?: P): UseIonModalResult;
export declare function useIonModal(component: React.ReactElement, componentProps?: any): UseIonModalResult;

Describe Alternatives

Leaving as-is

Related Code

No response

Additional Information

If you would accept this change, I would happily make a PR

Guia do colaborador

Direção de pesquisa
Modifique a definição de tipo para useIonModal no pacote React, adicionando parâmetro genérico e sobrecargas.
Pilha de tecnologia
typescriptreact
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScriptReactIonic Framework basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80