jsx-eslint/eslint-plugin-react

New Rule: Prevent boolean values for unknown props

Open

#1.695 geöffnet am 15. Feb. 2018

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.797 Forks)batch import
enhancementhelp wantedreact 16

Repository-Metriken

Stars
 (8.630 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

React 16 does not ignore Boolean values for unknown props anymore.

So while this code "worked" in react 15:

<div onClick={!disabled && onClick}/>

It would try to assing false to the DOM elements onClick handler when disabled === true (which would be ignored by the browser).

React 16 will print an error message in the console though.

How about a rule that does not allow boolean results for unknown props?

See https://reactjs.org/blog/2017/09/08/dom-attributes-in-react-16.html#changes-in-detail "Non-boolean attributes with boolean values"

Contributor Guide

Research-Richtung
Implementieren Sie eine neue ESLint Regel, die boolesche Werte erkennt, die unbekannten DOM Props in JSX zugewiesen werden, basierend auf der Verhaltensänderung von React 16. Verwenden Sie den AST, um JSXExpressionContainer auf boolesche Literale zu prüfen und gegen eine Liste bekannter Props zu validieren.
Tech Stack
javascriptreact
Domain
frontendtooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ESLintReact
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75