Skip to content

feat:それっぽいルーム表示カード - #58

Merged
sahara2006 merged 16 commits into
mainfrom
feat/makeRoomCard
Aug 26, 2026
Merged

feat:それっぽいルーム表示カード#58
sahara2006 merged 16 commits into
mainfrom
feat/makeRoomCard

Conversation

@sahara2006

Copy link
Copy Markdown
Contributor

#32
データの受け渡しがよくわかんなかった

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Issue #32 に対応して、ポータル内の部屋一覧で表示する「部屋カード(RoomCard)」コンポーネントを新規追加し、fixture データで見た目と動作(開閉・開始アクション)を確認できるようにする PR です。

Changes:

  • RoomCard コンポーネントを追加し、開閉可能な詳細表示と開始ボタンのイベント送出を実装
  • Histoire 用の story と、開閉・emit を確認するユニットテストを追加
  • トグルアイコン用に Material Symbols Outlined フォントを読み込む CSS を追加

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
client/src/RoomCard.vue Headless UI の Disclosure を使った部屋カード UI と start イベントを追加
client/src/RoomCard.story.vue fixture を使った Closed/Open の表示確認用 story を追加
client/src/assets/main.css Material Symbols Outlined のフォント読み込みを追加
client/src/tests/RoomCard.spec.ts 開閉動作と start emit を検証するテストを追加
Suppressed comments (1)

client/src/tests/RoomCard.spec.ts:30

  • ここも wrapper.get('button') だと将来 DOM 内のボタンが増えた際に違う要素をクリックしてしまう可能性があります。ヘッダー (.room-card__header) を明示的にクリックするようにしてください。
    await wrapper.get('button').trigger('click')
    expect(wrapper.get('.room-card__description').text()).toBe(room.description)

    await wrapper.get('button').trigger('click')
    expect(wrapper.find('.room-card__body').exists()).toBe(false)

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread client/src/RoomCard.vue Outdated
Comment thread client/src/assets/main.css Outdated
Comment thread client/src/__tests__/RoomCard.spec.ts
@sahara2006 sahara2006 self-assigned this Aug 26, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.

Suppressed comments (1)

client/src/RoomCard.vue:121

  • トグルは なので font-variation-settings / font-size / color などのフォント前提スタイルが効きません(将来の混乱の元になります)。画像のサイズ指定など、 に効くスタイルに置き換えるのがよいです。
.room-card__toggle {
  color: #aeb7d1;
  font-size: 1.25rem;
  font-variation-settings:
    'FILL' 0,

Comment thread client/src/RoomCard.vue
Comment thread client/src/assets/main.css Outdated
Comment thread client/src/RoomCard.vue Outdated

@kaomojikun5 kaomojikun5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

2点修正お願いします。
それとfigmaにあるやつみたいにライトテーマに合わせてくれると嬉しいです。

Comment thread client/src/__tests__/RoomCard.spec.ts
Comment thread client/src/RoomCard.vue
@sahara2006
sahara2006 requested a review from kaomojikun5 August 26, 2026 13:32

@kaomojikun5 kaomojikun5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

いいんじゃないでしょうか

@sahara2006
sahara2006 merged commit c6365ec into main Aug 26, 2026
10 checks passed
@sahara2006
sahara2006 deleted the feat/makeRoomCard branch August 26, 2026 13:41
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