Skip to content

Conversation

@Qs-F
Copy link
Contributor

@Qs-F Qs-F commented Oct 20, 2025

関連URL

https://kufuinc.slack.com/archives/CDG1XRPTP/p1759479243679429

概要

  • FormControl, FieldsetのexampleMessageをi要素でマークアップするように変更し、日本語では強調・区別には斜体を使う文化はないため、italicを外した

変更内容

  • FormControl, FieldsetのexampleMessageをi要素でマークアップするように変更
  • 日本語では強調・区別には斜体を使う文化はないため、italicを外した
  • italicをfalseにできるようにするため、Textを修正
    • tailwind-variantsの制約によりText[italic=false]とText[italic=undefined]の区別をつけることができずすべてのTextにshr-not-italicがついてしまうのを回避しinheritするため、ワークアラウンドでの修正

懸念点

exampleMessageをp>iで実装しているが、直接iでもいいような気もする

確認方法

@Qs-F Qs-F changed the title fix(FormControl, Fieldset): exampleMessageをi要素かつitalicで表示しないよう修正 fix(FormControl, Fieldset): exampleMessageをi要素かつ立体で表示 Oct 20, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 20, 2025

Open in StackBlitz

npm i https://pkg.pr.new/kufu/smarthr-ui@5902

commit: d40cf5c

@Qs-F Qs-F marked this pull request as ready for review October 21, 2025 00:56
@Qs-F Qs-F requested a review from a team as a code owner October 21, 2025 00:56
@Qs-F Qs-F requested review from AtsushiM and oti and removed request for a team October 21, 2025 00:56
@Qs-F
Copy link
Contributor Author

Qs-F commented Oct 21, 2025

@oti 以前話してたexampleMessageのi要素対応入れたマークアップの修正してるんで見てもらえると嬉しいです!

Comment on lines +109 to +110
/** 斜体にするかどうかの真偽値 */
italic?: boolean
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

italic にしたい時は className で渡すでも良さそうに思って、as="i" の時のデフォルトスタイルを定義しておくのはどうかしら。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あ〜 いいかもしれないですね、日本語主体のSmartHRではitalicはそもそも使わないはずという立場にたてる気がしたので良い気がしました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants