chore: decoratorsに対する処理を共通化できるよう、useDecoratorsを定義する #5326
Conversation
yagimushi
force-pushed
the
chore-add-use-decorators
branch
7 times, most recently
from
93b6e65 to
2f643da
Compare
yagimushi
force-pushed
the
chore-add-use-decorators
branch
from
2f643da to
0d284e1
Compare
AtsushiM
requested review from
misako0927 and
s-sasaki-0529
and removed request for
a team
commit: |
| @@ -1,7 +1,7 @@ | |||
| import React, { FC, KeyboardEventHandler, useCallback, useMemo } from 'react' | |||
| import { tv } from 'tailwind-variants' | |||
|
|
|||
| import { DecoratorsType } from '../../types' | |||
| import { type DecoratorsType } from '../../libs/decorator' | |||
There was a problem hiding this comment.
type の場合は type を強制する lint ってないのかしら。
There was a problem hiding this comment.
ありそう。
全プロダクトonにしてもいいのでは？ (可読性かなり上がりそう)
https://typescript-eslint.io/rules/consistent-type-imports/
| @@ -21,6 +21,30 @@ export default { | |||
| <Textarea {...args} maxLetters={5} value="テキスト" /> | |||
| <Textarea {...args} maxLetters={5} value="テキストエ" /> | |||
| <Textarea {...args} maxLetters={5} value="テキストエリア" /> | |||
| <Textarea | |||
There was a problem hiding this comment.
imo: decorator.ts の動作が担保されているのであれば、各コンポーネントで VRT に足す必要はなさそうに思いました。
There was a problem hiding this comment.
decorator.ts の動作が担保されているのであれば、各コンポーネントで VRT に足す必要はなさそうに思いました。
decorator自体の動作はそうですが、decoratorがどこに反映されるのか？は確認できないのでVRTに含めておくほうが良くないです？
特に今回のような処理が外部で行われる様になる場合はバグったことが検知しにくくなるのでテストしておきたい所存
| export type DecoratorType = (text: string) => ReactNode | ||
|
|
||
| export const useDecorators = <T extends string>( | ||
| defaultTexts: { [K in T]: string }, |
There was a problem hiding this comment.
imo: デフォルトテキスト、というかは変換表だなと思った。translateTable 的な。
There was a problem hiding this comment.
変換表
いや、指定無い場合の文字列のまとまりなんでtranslateTableだと役割の一部しか表せてない気がします
役割も翻訳だけじゃなくて違う文字列にすることも有り得るのでtranslateは不適切っぽい。
…e-add-use-decorators
yagimushi
force-pushed
the
chore-add-use-decorators
branch
4 times, most recently
from
7c70320 to
606ccb4
Compare
yagimushi
force-pushed
the
chore-add-use-decorators
branch
from
606ccb4 to
7f05277
Compare
関連URL
概要
変更内容
確認方法