Skip to content

Conversation

akagane99
Copy link
Contributor

@akagane99 akagane99 commented Mar 14, 2025

概要

  • /file/css/1.css系がブラウザキャッシュに対応してなかったため、処理を見直し+ブラウザキャッシュに対応しました。
  • /file/css/1.css系を2つに別けました。
    • サイト全体の動的css:/file/css/site.css
    • ページ毎の動的css:/file/css/page/1.css
    • CSSの読み込み順は1.サイト全体CSS、2.ページ毎CSS です。
      これにより、同じcss classがどちらにあった場合、「2.ページ毎CSS」が勝ちます。(2つに別ける前と同じ動き)
  • ブラウザキャッシュしても、設定更新時には新しいCSSを読み込ませるため、上記にクエリストリング(?version=999999、値は例)をそれぞれ追加しました。
  • 不具合修正
    • サイト全体CSSに含まれている「カテゴリーCSS出力」に、同じcss classが設定されていても構わず出力する不具合があったため、重複したcss classは出力しないよう修正しました。

github actionsでテスト

Laravel Dusk Connect-cms-test matrix · db4ea2d
https://github.com/opensource-workshop/connect-cms/actions/runs/13889880636
→ エラーなし

修正後画面

トップページ

image
→ 2回目の表示以降、ブラウザキャッシュを確認(/file/css/site.css, /file/css/page/3.css)

ログイン画面

image
→ site.cssのブラウザキャッシュを確認(/file/css/site.css)
→ /file/css/page/1.css系は読み込んでない事を確認

管理者メニュ-画面

image
→ site.cssのブラウザキャッシュを確認(/file/css/site.css)
→ /file/css/page/1.css系は読み込んでない事を確認

レビュー完了希望日

なし

関連Pull requests/Issues

参考

DB変更の有無

無し

チェックリスト

@akagane99 akagane99 changed the title change: コア, /file/css/1.css系のブラウザキャッシュ対応 [コア] /file/css/1.css系のブラウザキャッシュ対応 Mar 14, 2025
@akagane99 akagane99 added the enhancement 機能強化 label Mar 14, 2025
@akagane99 akagane99 self-assigned this Mar 14, 2025
@akagane99 akagane99 marked this pull request as ready for review March 14, 2025 11:24
@akagane99 akagane99 merged commit 1b860c7 into master Mar 17, 2025
4 checks passed
@akagane99 akagane99 deleted the file-css branch March 17, 2025 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能強化
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant