Changelog エントリの統合と不要コメントの除去
この PR は <wa-button> の Firefox アイコン位置不具合に関する記述を、個別コンポーネントのドキュメントからリポジトリ全体の changelog へ移動し、同時に対応コード内の説明コメントを削除します。
背景
packages/webawesome/docs/docs/resources/changelog.md に個別コンポーネントの修正履歴が散在していたため、変更履歴の一元管理が難しくなっていました。特に <wa-button> のバグ修正は #2426 で報告されており、同じ情報がコンポーネント固有のドキュメントと全体の changelog に重複記載されていました。重複した記述はメンテナンスコストを増大させ、リリースノート生成時に情報の整合性を保つのが困難になるという課題がありました。
技術的な変更
ドキュメント側の移行
docs/resources/changelog.md に以下の行が 追加 され、 <wa-button> のバグ修正エントリが 削除 されました。
@@ -22,10 +22,10 @@ These are still finding their shape. APIs can change between minor versions, so
- Fixed a bug in `<wa-checkbox>` and `<wa-switch>` where `.checked` property would not properly update the shadow dom checkbox.
- Fixed a bug where `@lit-labs/ssr` was not included as a dependency.
+- Fixed a bug in `<wa-button>` where the slotted icon was rendered off-center at `size="xs"` in Firefox [issue:2426]
:::
-
## 3.8.0
この変更により、<wa-button> の問題はリポジトリ全体の変更履歴に統合され、他のコンポーネントと同様の参照形式で記載されます。
ソースコード側の整理
packages/webawesome/src/components/button/button.styles.ts から、 Firefox のアイコン位置不具合に関するコメントが 4 行削除 されました。削除前は次のようになっていました。
@@ -279,10 +279,6 @@ export default css`
.is-icon-button .label {
display: flex;
- /* Center the slotted icon horizontally so the icon sits in the
- middle of the square label regardless of the icon host's
- intrinsic width. Without this, Firefox renders the icon off-
- centre at small sizes (e.g. size="xs"). */
justify-content: center;
}
コメントはバグ修正がドキュメントに明示されたことで冗長になると判断され、機能自体には影響しない単なる説明文として除去されました。
設計判断
changelog の一元化 は、リリースノート生成や変更履歴検索時の一貫性を保つための設計方針です。個別ドキュメントに散在していたエントリを集約することで、変更点の把握が容易になり、将来的なメンテナンスコストが削減されます。また、コードコメントの削除 は、ドキュメントに説明が十分に存在する場合のクリーニング作業として位置付けられ、実装上の振る舞いには影響しません。これにより、コードベースは実装ロジックに集中し、余計なテキスト情報は外部ドキュメントに委譲される形となります。
まとめ
本 PR は <wa-button> の Firefox アイコン位置不具合に関する情報をリポジトリ全体の changelog に統合し、同時にソースコード内の不要な説明コメントを除去することで、変更履歴の一貫性とコードベースのクリーンさを向上させました。