@attr boolean/number setter function called with an empty string (v2-beta)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- typescript
- Domaine
- frontend
Piste de recherche
Commencez au point d’entrée du décorateur @attr et reproduisez le comportement avec le composant TypeScript et les extraits HTML de l’issue. Suivez les appels au setter pour un attribut boolean-prop manquant et comparez-les aux appels lorsqu’une valeur initiale est présente. Le travail est terminé lorsque le comportement attendu du setter est établi et couvert par un test approprié, ou documenté comme une limitation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
This is maybe related to #117, but not 100% sure on that.
With a component defined like this:
import { controller, attr } from '@github/catalyst'
@controller
class TestComponentElement extends HTMLElement {
@attr get booleanProp() {
return false
}
set booleanProp( _v: boolean ) {
console.dir( `booleanProp type: ${ typeof _v }` )
console.dir( `booleanProp value: '${ _v }'` )
}
connectedCallback() {
this.booleanProp = true
}
}
and html like this (note the lack of an initial boolean-prop value):
<test-component></test-component>
I get these log messages:
booleanProp type: boolean
booleanProp value: 'true'
booleanProp type: string
booleanProp value: ''
Changing the html to include an initial value like this:
<test-component boolean-prop='anything'></test-component>
get's rid of that final empty string call:
booleanProp type: boolean
booleanProp value: 'true'
Maybe this is a limitation? Hopefully this isn't another ghost bug, sorry about the last one 🫣
- Langage dominant
- TypeScript
- Étoiles
- 1.4k
- Forks
- 61
- Merge moyen
- 1 j 15 h
- PR mergées (30 j)
- 8
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de github/catalyst
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 62/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
-
EcmaScript DecoratorsOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
Toutes les issues de github/catalyst
Issues similaires
-
normalizePropName strips `#` characters from Figma property names instead of only the `#id` suffixOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
figma/code-connect#419 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
jeanibarz/knowledge-base-mcp-server#1268 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
Koenkk/zigbee-herdsman-converters#13820 ·
Les mainteneurs répondent en général sous 2 jours
-
A live recovery copy treats a large whole number and text with the same digits as the same valueOuvertebug sharing & sync
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
Les mainteneurs répondent en général sous 1 jour
-
[prefer-expect-resolves] autofix leaves a stray ")" when the awaited expression is parenthesizedOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
jest-community/eslint-plugin-jest#2052 ·