Feat i18n support for DatePicker and DateRangePicker #1398
Conversation
🦋 Changeset detectedLatest commit: d742867 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
✅ Deploy Preview for ingred-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| /** | ||
| * カレンダーに表示する曜日のリスト | ||
| * @memo dayjs().format("ddd") で対応したいが、階層が深くなったりするので一旦静的な値で対処 | ||
| */ | ||
| weekList?: string[]; |
There was a problem hiding this comment.
曜日一覧も props から受け取るようにした。
dayjs を使えば指定した locale で曜日を取得することもできるけど、階層が深くなることでどこで何を扱っているかが不明瞭になったりするので、一旦 props 経由で受け取り上から流すだけの構成にする。
この PR では対応しないだけで、後でいい構成に変更するかも。
takurinton
force-pushed
the
feat-i18n-datepicker
branch
from
3ffd257 to
d742867
Compare
|
Calendar コンポーネントの粒度でもやった方がいい気がしつつ、単体で使うユースケースがないため一旦やってない。 |
| type Props = { | ||
| date: Dayjs; | ||
| monthFormat?: string; | ||
| weekList?: string[]; |
There was a problem hiding this comment.
率直な疑問ですが、この辺のフォーマットは現状型を強く縛らない方針なのでしょうか？
言語の違いはあれど扱う文字列は決まってそうなのでユニオン等で表現するのもありかなと思いまして。
この辺の話で一旦リテラル型にしているのか
/**
* カレンダーに表示する曜日のリスト
* @memo dayjs().format("ddd") で対応したいが、階層が深くなったりするので一旦静的な値で対処
*/There was a problem hiding this comment.
ここは useLocaleProps を通じて曜日のリストを管理するための暫定的なものであり、実質ユーザーから渡される想定ではないのであまり強く縛る必要もないかなと思ってこのようにしてます！
たしかに英語と日本語くらいしか対応するつもりもないので言われてみたら縛ってもいいかな？と思ったのですが、一旦このままでも良いかなとも思います。
#975