Attrs initialization without `@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
- Domaine
- frontend
Piste de recherche
Commencez par lire src/attr.ts autour du guard initialisé et src/core.ts, où controller enveloppe connectedCallback(). Reproduisez l’exemple InfoMessage avec initializeAttrs(this, ['open']) et data-open="false". C’est terminé lorsque l’initialisation manuelle sans @attr applique la valeur de l’attribut au lieu de laisser open à true.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hi there 😄
After upgrading from v1.1.4 to v1.3.2, I noticed that my attributes were no longer shadowed by initializeAttrs(). I started digging and stumbled upon this piece of logic introduced in #191:
https://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/attr.ts#L39
It prevents initializeAttrs() from doing anything the second time around, thus skipping the manual initialization of attributes. Also, it seems like this change was one of the main things addressed by the above-mentioned PR.
So, what would be the correct way now to initialize the attributes without the @attr decorator?
Thanks!
Example
<script>
class InfoMessage extends HTMLElement {
open = true
connectedCallback() {
initializeAttrs(this, ['open'])
}
}
controller(InfoMessage)
</script>
<info-message data-open="false"></info-message>
controller(InfoMessage)wraps originalconnectedCallback()and calls:
https://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/core.ts#L14
InfoMessageis marked asinitialized(attrs.ts):
https://github.com/github/catalyst/blob/a8fb3ba59a4ac258d0fe64a50ecd3b6997f5f016/src/attr.ts#L40
- The original
connectedCallback()is executed, butinitializeAttrs()would hit that early-return condition. this.openis stuck in its default state (true) 😔
- 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 ·