タイムテーブルを表示できるようにする #79
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
モバイル版の修正が終わるまでDraftにする。 |
- HTMLファイル内のJavaScript検証コードを削除 - HTML Prooferのカスタムテストとして重複検知を実装 - 重複があれば CI (bundle exec rake test) でお知らせ 本番環境に手動で確認するテスト用 JavaScript コードを含めずに、 既存の CI/CD で自動的に実行および検証・確認できるようにしました。
…ator refactor: タイムテーブル重複検知を CI 上で動かせるテストに移行
- Liquidテンプレートから時間計算をプラグインとして実装して簡略化 - Liquidテンプレートから複雑なロジックを除去し、表示のみに専念 - 時間計算のロジックと、描画のテンプレートの可読性をそれぞれ改善
- プラグインコードを30%削減（145行→91行） - メソッド数を57%削減（7個→3個） - grid→tableで一貫性のある命名に統一 - 保守性と可読性が大幅に向上 - ビルドパフォーマンスも改善 - コードベース全体で一貫性のある設計
- span → duration: イベントの継続時間を表す適切な名前に変更 - start_slot/end_slot → slot_start/slot_end: 命名パターンを統一 - start_minutes/end_minutes → event_start/event_end: イベントの時間であることを明確化 他、細かな cosmetic change など。
- プラグインから未使用のrequire文を削除 - Gemfileからactivesupport gemを削除 - bundle installで依存関係を元の状態に戻した
…plugin refactor: タイムテーブルの計算処理と描画処理を分離し、それぞれの役割を明確化
- テーブル内部での縦スクロールを無効化 - (他ページ同様の) 縦スクロールに変更
Merged
…play fix: タイムテーブルのモバイル表示を改善
mucho613
reviewed
Sep 10, 2025
_pages/time-table.html
Outdated
Comment on lines
27
to
29
| <caption> | ||
| {{ site.date_event }} のタイムテーブル | ||
| </caption> |
There was a problem hiding this comment.
イベントは1日なので、日付の表示は不要そうです 🧹
Suggested change
| <caption> | |
| {{ site.date_event }} のタイムテーブル | |
| </caption> |
There was a problem hiding this comment.
847d7ea こちらで日付情報を消去しました。
mucho613
reviewed
Sep 10, 2025
mucho613
reviewed
Sep 10, 2025
|
time_table.yml の内容を整理しました |
Fukusan64
reviewed
Sep 10, 2025
Fukusan64
reviewed
Sep 11, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
現状は、PCのみのレイアウト調整を行った。スマホのレイアウトを変更しようと思ったが、PC/Mobile 両方とも対応済み ✅中会議室（WS２）LTブースまでしか横がスクロールすることができなく修正することができなかったので保留している。Action List
/time-table/ページで描画中会議室（WS4）💻 PC 画面
https://gyazo.com/b56f8caa7f524dc409b7110c4ad2be8b
📱 Mobile 画面
https://gyazo.com/0fb34b84d2094a4d4233abd8624a10ff