@attr boolean/number setter function called with an empty string (v2-beta)
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 45/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- typescript
- 領域
- frontend
調査の方向性
@attr デコレーターのエントリーポイントから開始し、issue 内の TypeScript コンポーネントと HTML スニペットを使って動作を再現します。欠落している boolean-prop 属性に対する setter 呼び出しを追跡し、初期値が存在する場合の呼び出しと比較します。期待される setter の動作が確立され、適切なテストでカバーされるか、制限事項として文書化されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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 🫣
- 主要言語
- TypeScript
- スター
- 1.4k
- フォーク
- 61
- 平均マージ
- 1日 15時間
- マージ済み PR（30日）
- 8
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/catalyst のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 62/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
github/catalyst の issue をすべて見る
似ている issue
-
normalizePropName strips `#` characters from Figma property names instead of only the `#id` suffixオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
figma/code-connect#419 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
jeanibarz/knowledge-base-mcp-server#1268 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
Koenkk/zigbee-herdsman-converters#13820 ·
メンテナーはふだん 2 日以内に返信
-
A live recovery copy treats a large whole number and text with the same digits as the same valueオープンbug sharing & sync
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
メンテナーはふだん 1 日以内に返信
-
[prefer-expect-resolves] autofix leaves a stray ")" when the awaited expression is parenthesizedオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
jest-community/eslint-plugin-jest#2052 ·