Project

General

Profile

Enhancement(機能追加・改善) #3742

スマートフォン用の CSS ファイルが OpenPNE 本体側とテーマファイルで二重に読み込まれるのを防止する

Added by Rimpei Ogawa over 7 years ago. Updated over 7 years ago.

Status:
Pending Review(レビュー待ち)
Priority:
Normal(通常)
Assignee:
Target version:
-
Start date:
2014-12-10
Due date:
% Done:

50%


Description

#3740 前提

スマートフォン用の CSS ファイルが OpenPNE 本体側とテーマファイルで二重に読み込まれるのを防止する。
二重に読み込まれてもデザイン上は問題なく表示できる可能性が高いですが、無駄な処理なのは間違いないです。

本体側がハードコーディングで手が出せないので、プラグイン側にレイアウトのテンプレートファイルを置いて上書きする方針。
元ファイルがカスタマイズされている場合(※)があるので、タスクで自動的に変換するようにします。
(※例えば、 opTimelinePlugin のパッチ を適用している場合など)


タスクのヘルプ

$ ./symfony help opSkinThemePlugin:build-smt-layout
Usage:
 symfony opSkinThemePlugin:build-smt-layout

Description:
 スマートフォン用の CSS ファイルが二重に読み込まれるのを避けるため、
 OpenPNE 本体側のレイアウトテンプレートの以下の記述を削除したファイルを生成します。

   <?php op_smt_use_stylesheet('...') ?>

 apps/pc_frontend/templates/smtLayout*.php
 ↓
 plugins/opSkinThemePlugin/apps/pc_frontend/templates/smtLayout*.php

 バージョンアップ、カスタマイズ等により、OpenPNE 本体側のレイアウトテンプレートを
 変更した際には再度このタスクを実行してください。

Related issues

Related to opSkinThemePlugin - Enhancement(機能追加・改善) #3740: スマートフォン用に css, js ファイルを読み込めるようにする Pending Review(レビュー待ち) 2014-12-10

History

#1 Updated by Rimpei Ogawa over 7 years ago

  • Description updated (diff)
  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

スマートフォン用の CSS ファイルが OpenPNE 本体側とテーマファイルで二重に読み込まれるのを防止するためのタスクを追加 (refs #... by balibali · Pull Request #16 · tejimaya/opSkinThemePlugin
https://github.com/tejimaya/opSkinThemePlugin/pull/16

で Pull Request しました。

#2 Updated by Rimpei Ogawa over 7 years ago

#3 Updated by Rimpei Ogawa over 7 years ago

https://github.com/tejimaya/opSkinThemePlugin/pull/16
許可をもらってマージしました。

レビュー、テストはまだなのでステータスはレビュー待ちのままにしておきます。

Also available in: Atom PDF