Attrs are never observed unless added with `@attr` decorator
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- typescript
Piste de recherche
Commencez par src/attr.ts, en particulier defineObservedAttributes() et getAttrNames(), puis comparez leur comportement avec l’exemple vanilla de docs/_guide/attrs.md. Vérifiez si les attributs initialisés manuellement peuvent être observés sans @attr ; le travail est terminé lorsque l’implémentation et le guide concordent sur la manière de spécifier les attributs observés.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hi folks,
While I was testing the attributes — trying to work my way around #207 — I found another subtle detail. Manually initialized attributes are never observed.
Catalyst's guide provides the following example for vanilla users: https://github.com/github/catalyst/blob/main/docs/_guide/attrs.md#what-about-without-decorators.
Please notice the second argument there defineObservedAttributes(HelloWorldElement, ['foo']) and compare it with the actual function declaration:
https://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/attr.ts#L99
It doesn't expect the second argument (arguments aren't used either).
Now, internally defineObservedAttributes() pulls attribute names from getAttrNames(), which, in turn, pulls them from attrs (WeakMap). The only way to populate attrs is to use @attr decorator:
https://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/attr.ts#L15
Maybe I'm missing something, but it seems like there's no way to specify observed attributes without decorators. In any case, it'd be very nice to clarify that 🙂
Thank you!
- 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 ·