Skip to content

Conversation

@hikahana
Copy link
Contributor

@hikahana hikahana commented Mar 22, 2025

対応Issue

resolve #1780

概要

  • リスト表記にする部分をコンポーネントにしました。

実装詳細

  • ない
  • figmaのコードパクってgptに投げて整形した。
  • figma relativeで表現されてたのはpadding-leftで対応したから多少の誤差あるかも、、、、
  • propsで渡す際の型であるFormItemはexport constで定義しているのでどこからでも引っ張てこれる

画面スクリーンショット等

image

テスト項目

  • コードレビューして。案出ない
  • デザインとの差異がないか
  • モバイル、PCでよしなにしてるか

備考

変更できません という部分があったがその実装は大変めんどう、、、
配列の要素をさらに追加する???

急ピッチで作成したのでpropsで渡すときは以下の形になっちゃう。。。
よしなに修正したいのですが、案あれば教えてください。。。。

const items: FormItem[] = [
  { label: '団体名', content: content.hoge },
  { label: 'hello', content: content.test },
];

@hikahana hikahana self-assigned this Mar 22, 2025
Copy link
Collaborator

@izuizu0424 izuizu0424 left a comment

Choose a reason for hiding this comment

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

デザインはPC,モバイル共にOKでした
急なissue対応ありがとうございます

@TkymHrt TkymHrt self-requested a review April 3, 2025 07:01
Copy link
Member

@TkymHrt TkymHrt left a comment

Choose a reason for hiding this comment

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

動作OKでした。コメントしています!

Comment on lines 1 to 4
export type FormItem = {
label: string;
content: string;
};
Copy link
Member

Choose a reason for hiding this comment

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

とりあえず変更不可のプロパティを設定しておけば、表示は別途コンポーネント側で対応できるのかなと思いました。

Suggested change
export type FormItem = {
label: string;
content: string;
};
export type FormItem = {
label: string;
content: string;
editable?: boolean;
};
{item.editable === false && (
   <span className="text-red-500 ml-6 text-xs">※変更できません</span>
)}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

今日のgm2 mtで決まっただけど

ラベルずを作る


label.ts で,定数を宣言しておく.

こんな感じでlabel部分の定数を置いておいて、labelとdataを結合してFormItemで渡すことになったから、この修正ちょっと厳しいかな?
https://www.notion.so/nutfes-nutmeg/GM2-1cb41f19206380b682d9e6cf7fc710b5?pvs=4

Copy link
Contributor Author

Choose a reason for hiding this comment

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

よく考えたらオプショナルだし、結合の時に一緒にやればいけるのか。
要確認でいきます!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

これ変更できないパターンの方が少ないから逆にします

@hikahana hikahana requested a review from TkymHrt April 7, 2025 13:32
Copy link
Member

@TkymHrt TkymHrt left a comment

Choose a reason for hiding this comment

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

LGTM

@hikahana hikahana merged commit 3f84f72 into gm3/develop Apr 10, 2025
@hikahana hikahana deleted the feat/hikahana/1780-add-request-list-view branch April 10, 2025 07:49
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.

申請項目のリスト表示部分コンポーネント作成

5 participants