refactor: タイムテーブルの計算処理と描画処理を分離し、それぞれの役割を明確化 #81
Merged
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
- Liquidテンプレートから時間計算をプラグインとして実装して簡略化 - Liquidテンプレートから複雑なロジックを除去し、表示のみに専念 - 時間計算のロジックと、描画のテンプレートの可読性をそれぞれ改善
- プラグインコードを30%削減（145行→91行） - メソッド数を57%削減（7個→3個） - grid→tableで一貫性のある命名に統一 - 保守性と可読性が大幅に向上 - ビルドパフォーマンスも改善 - コードベース全体で一貫性のある設計
yasulab
changed the title
yasulab
changed the title
yasulab
changed the title
yasulab
changed the title
- 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で依存関係を元の状態に戻した
|
@takusandayooo こちらもし良さそうならマージしていただけると...!! 🙏✨ |
|
@yasulab |
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.
概要
タイムテーブルの計算処理（文字列処理・時刻計算など）をLiquidテンプレートから分離し、それぞれの役割を明確化しました。
1. 計算処理をテンプレートから分離 (d662724)
_plugins/time_table_generator.rbでデータを計算2. 計算結果をテンプレートに渡して描画 (2d26d3e)
_data/time_table.ymlのデータ構造をそのまま活用して計算分離後のそれぞれの役割
_plugins/time_table_generator.rb: タイムテーブルのイベント計算部分を担当_pages/time-table.html: タイムテーブルのイベント描画部分を担当関連 PRs