Skip to content

タイムテーブルを表示できるようにする #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

タイムテーブルを表示できるようにする #79

wants to merge 16 commits into from
+493 −0

Conversation

takusandayooo
Copy link
Collaborator

@takusandayooo takusandayooo commented Sep 9, 2025 •
edited by yasulab
Loading

現状は、PCのみのレイアウト調整を行った。スマホのレイアウトを変更しようと思ったが、中会議室（WS２）LTブースまでしか横がスクロールすることができなく修正することができなかったので保留している。 PC/Mobile 両方とも対応済み ✅

Action List

💻 PC 画面

b56f8caa7f524dc409b7110c4ad2be8b
https://gyazo.com/b56f8caa7f524dc409b7110c4ad2be8b


📱 Mobile 画面

0fb34b84d2094a4d4233abd8624a10ff
https://gyazo.com/0fb34b84d2094a4d4233abd8624a10ff

@takusandayooo takusandayooo marked this pull request as draft September 9, 2025 06:33
@takusandayooo
Copy link
Collaborator Author

モバイル版の修正が終わるまでDraftにする。

@yasulab
タイムテーブル重複検知をカスタムテストに移行してCIに追加
2ec6f74 
- HTMLファイル内のJavaScript検証コードを削除
- HTML Prooferのカスタムテストとして重複検知を実装
- 重複があれば CI (bundle exec rake test) でお知らせ

本番環境に手動で確認するテスト用 JavaScript コードを含めずに、
既存の CI/CD で自動的に実行および検証・確認できるようにしました。
@yasulab yasulab mentioned this pull request Sep 10, 2025
Merged
@yasulab
Merge pull request #80 from coderdojo-japan/refactor-time-table-valid…
86e39e9 
…ator

refactor: タイムテーブル重複検知を CI 上で動かせるテストに移行
@yasulab yasulab self-requested a review September 10, 2025 06:44
@yasulab
refactor: タイムテーブルのロジックをJekyllプラグインに移行
d662724 
- Liquidテンプレートから時間計算をプラグインとして実装して簡略化
- Liquidテンプレートから複雑なロジックを除去し、表示のみに専念
- 時間計算のロジックと、描画のテンプレートの可読性をそれぞれ改善
@yasulab
refactor: タイムテーブルをイベント中心設計にして一貫性を強化
2d26d3e 
- プラグインコードを30%削減（145行→91行）
- メソッド数を57%削減（7個→3個）
- grid→tableで一貫性のある命名に統一

- 保守性と可読性が大幅に向上
- ビルドパフォーマンスも改善
- コードベース全体で一貫性のある設計
@yasulab yasulab mentioned this pull request Sep 10, 2025
Merged
@yasulab
コメントの細かな用語を統一
95dd2f0
@yasulab
refactor: 変数名を意味的に明確化
a77d223 
- span → duration: イベントの継続時間を表す適切な名前に変更
- start_slot/end_slot → slot_start/slot_end: 命名パターンを統一
- start_minutes/end_minutes → event_start/event_end: イベントの時間であることを明確化

他、細かな cosmetic change など。
@yasulab
chore: 不要なActive Support依存を削除
4c5e3c0 
- プラグインから未使用のrequire文を削除
- Gemfileからactivesupport gemを削除
- bundle installで依存関係を元の状態に戻した
@yasulab yasulab removed their request for review September 10, 2025 10:46
takusandayooo and others added 3 commits September 10, 2025 19:54
@takusandayooo
Merge pull request #81 from coderdojo-japan/refactor-time-table-with-…
80e7720 
…plugin

refactor: タイムテーブルの計算処理と描画処理を分離し、それぞれの役割を明確化
@Fukusan64 @yasulab
色々調整してみた(ナビバーと相性が悪い...)
dc7d66c
@yasulab
fix: タイムテーブルの横スクロールのみ可能に変更
3bf2ce7 
- テーブル内部での縦スクロールを無効化
- (他ページ同様の) 縦スクロールに変更
@takusandayooo takusandayooo self-assigned this Sep 10, 2025
@yasulab yasulab mentioned this pull request Sep 10, 2025
Merged
@takusandayooo
Merge pull request #83 from coderdojo-japan/fix-time-table-mobile-dis…
5f624da 
…play

fix: タイムテーブルのモバイル表示を改善
mucho613
mucho613 reviewed Sep 10, 2025
View reviewed changes
mucho613
mucho613 reviewed Sep 10, 2025
View reviewed changes
mucho613
mucho613 reviewed Sep 10, 2025
View reviewed changes
@kp54
Copy link
Collaborator

kp54 commented Sep 10, 2025

time_table.yml の内容を整理しました

@yasulab yasulab mentioned this pull request Sep 10, 2025
Closed
Fukusan64
Fukusan64 reviewed Sep 10, 2025
View reviewed changes
@takusandayooo takusandayooo marked this pull request as ready for review September 11, 2025 01:40
@takusandayooo
Copy link
Collaborator Author

takusandayooo commented Sep 11, 2025 •
edited
Loading

@mucho613

今やらなくても良いと思うんですが、
もしかするとタイムテーブルの各イベント項目から、その告知ページ等に飛べたら便利かもな……と思いました

こちらですが、簡単そうだったので4214bbc ,ea321c8で対応しました。

Fukusan64
Fukusan64 reviewed Sep 11, 2025
View reviewed changes
Copy link
Collaborator

@Fukusan64 Fukusan64 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

雛形作成という意味ではLGTMです
(他の部分のコメントもあるのでそこもレビュー貰っておきたいです。 🙏)

自分の認識が合っていれば
確定版のタイムテーブルはまだ揃っていないはずなので
今の内容が本番に出てしまうのはマズいなと思っています。

残件としては

  • 正しいタイムテーブルの取得、反映
  • タイムテーブルページへの導線準備

このあたりかなと思います

@takusandayooo takusandayooo marked this pull request as draft September 11, 2025 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fukusan64 Fukusan64 Fukusan64 left review comments

@mucho613 mucho613 Awaiting requested review from mucho613

@kp54 kp54 Awaiting requested review from kp54

Assignees

@takusandayooo takusandayooo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@takusandayooo @kp54 @mucho613 @Fukusan64 @yasulab