Project

General

Profile

Actions

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

open

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

Added by Rimpei Ogawa about 10 years ago. Updated about 10 years ago.

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

50%

Estimated time:

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 1 (1 open0 closed)

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

Actions
Actions #1

Updated by Rimpei Ogawa about 10 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 しました。

Actions #2

Updated by Rimpei Ogawa about 10 years ago

Actions #3

Updated by Rimpei Ogawa about 10 years ago

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

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

Actions

Also available in: Atom PDF