palantir/blueprint

Alert doesn't support usePortal and portalClassName TS definitions, but they work fine

Open

#5801 aperta il 13 dic 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
Domain: APIP3Package: coreType: feature requesthelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

Environment

  • Package version(s): 4.13.0
  • Operating System: Windows 11
  • Browser name and version: Chrome latest

Steps to reproduce

Use the usePortal prop on a Blueprint TSX project. A red error message displays the message Property 'usePortal' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes<Alert> & Pick<Readonly<IAlertProps>, never> & InexactPartial<...> & InexactPartial<...>'..

The Blueprint docs at https://blueprintjs.com/docs/#core/components/alert and https://github.com/palantir/blueprint/blob/develop/packages/core/src/components/alert/alert.tsx don't mention these two props either. But forcing compilation works and it's clear from DevTools that usePortal works fine.

Guida contributor

Direzione di ricerca
Aggiungi le proprietà `usePortal` e `portalClassName` all'interfaccia TypeScript `IAlertProps`. Controlla altri componenti Blueprint come Dialog che già definiscono queste proprietà e copia il pattern.
Tech stack
typescriptreact
Dominio
frontend
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
TypeScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70