palantir/blueprint

Dev mode warning for Tooltip wrapping a disabled Button

Open

#3 494 ouverte le 23 avr. 2019

Voir sur GitHub
 (0 commentaires) (2 réactions) (0 assignés)TypeScript (20 263 stars) (2 167 forks)batch import
Domain: APIP2Package: coreType: enhancementhelp wanted

Description

Even though it's well-documented, the issue comes up frequently: https://github.com/palantir/blueprint/issues/967#issuecomment-485615346

We should log a warning in development mode, suggesting use of an AnchorButton instead. Writing a lint rule seems like it would be harder than introspecting the React tree at runtime.

cc @thelastnode

Guide contributeur

Stack technique
reacttypescript
Domaine
frontend
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ReactBlueprint componentsTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
Implement a dev mode warning in the Tooltip component that checks if its children include a disabled Button. Inspect the Tooltip component source in the Blueprint repository (likely in packages/core/src/components/tooltip). Use React.Children or React.Children.forEach to iterate over children and check if any element is a Button with disabled prop. If so, log a warning suggesting AnchorButton. Look at existing dev warnings in Blueprint (e.g., using the debug package or process.env.NODE ENV) to maintain consistency.