traPtitech / traPtitech/Checkin
payouts.vue の UAlert に渡している size が効いていない
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 0
- Forks
- 0
- Avg merge
- 3d 5h
- Merged PRs (30d)
- 7
Description
症状
apps/web/app/pages/payouts.vue は、行ごとの警告に使う 3 つの UAlert に alertFallthroughAttrs(中身は { size: 'sm' })を v-bind で渡している。@nuxt/ui 3.3.7 の Alert.vue は size を prop として宣言していない。2026-09-18 に node_modules に展開された @nuxt/ui 3.3.7 の dist/runtime/components/Alert.vue の props の宣言を読んだところ、並んでいるのは as・title・description・icon・avatar・color・variant・orientation・actions・close・closeIcon・class・ui で、size は無かった。
宣言されていない属性はフォールスルー属性として根の要素にそのまま出るので、この 3 箇所は素の size 属性として markup に現れるだけで、見た目を変えていない。
誰が何を誤るか
このファイルを読む人が、行ごとの警告は小さく描画されると読む。実際には他の UAlert と同じ大きさで出る。size を消すか、@nuxt/ui が用意している大きさの指定に置き換えるかを判断する材料が、コードからは読み取れない。
直さずに残した理由
削除は出力される markup の変更になるので、見た目の回帰を確かめる単位を分けた。@nuxt/ui 側が size を prop として宣言する版が出た場合は、削除ではなく宣言済み prop として書き直すのが正しい対応になるため、どちらを採るかは版の状況で変わる。
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with apps/web/app/pages/payouts.vue and inspect the three UAlert usages plus the installed @nuxt/ui 3.3.7 Alert.vue props. Confirm which sizing API that version supports, then choose whether to remove or replace the ineffective size binding and verify that the rendered alerts have the intended appearance without an unintended size attribute.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- nuxt, typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100