Skip to content

Conversation

@Fukusan64
Copy link
Collaborator

@Fukusan64 Fukusan64 commented Oct 17, 2025

やったこと

  • 九州Dojo座談会の記事を追加
  • みんなのセッションのdateを修正
  • 各種セッション詳細のサムネイル画像を全体的に(w-max-100 → w-max-150)少し大きく表示する
  • 登壇者、所属が存在しない場合にもTOPの表示が乱れないようif文を追加する
  • TOPの登壇者表記を読みやすく改善

記事追加については以下の通り
image
(TOPの登壇者情報の表示を改善しています)
image

TOPの登壇者情報の表示(SP)
image

@Fukusan64 Fukusan64 self-assigned this Oct 17, 2025
@Fukusan64 Fukusan64 marked this pull request as ready for review October 17, 2025 14:12
Comment on lines 68 to 70
speakers:
- name: (進行役)宮島衣瑛
affiliation: CoderDojo Japan 理事
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
speakers:
- name: (進行役)宮島衣瑛
affiliation: CoderDojo Japan 理事
speakers:
- name: (進行役)宮島衣瑛
affiliation:
- CoderDojo Japan 理事

Copy link
Collaborator Author

@Fukusan64 Fukusan64 Oct 18, 2025

Choose a reason for hiding this comment

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

kp54
kp54 previously approved these changes Oct 18, 2025
Copy link
Collaborator

@kp54 kp54 left a comment

Choose a reason for hiding this comment

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

LGTM ありがとうございます

@Fukusan64
Copy link
Collaborator Author

LGTM ありがとうございます

まだ修正してないです

@Fukusan64 Fukusan64 marked this pull request as draft October 18, 2025 07:09
@Fukusan64 Fukusan64 requested a review from kp54 October 18, 2025 07:21
@Fukusan64 Fukusan64 marked this pull request as ready for review October 18, 2025 07:21
@Fukusan64 Fukusan64 merged commit f3695da into main Oct 18, 2025
2 checks passed
@Fukusan64 Fukusan64 deleted the add-kyusyu-dojo-discussion branch October 18, 2025 07:27
@kp54
Copy link
Collaborator

kp54 commented Oct 18, 2025

LGTM ありがとうございます

まだ修正してないです

修正前の状態でもレンダリング結果は正しいので、より正確には配列であるべきだがスカラーでも支障はない、という性質のものでした。

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