Simplicityの特徴 › フォーラム › 要望・機能追加など › JavaScriptまわりの変更(head cleanerプラグインとの組み合わせ)
- このトピックには6件の返信、2人の参加者があり、最後に
koma2により10年前に更新されました。
-
投稿者投稿
-
-
2015年1月23日 8:07 PM #18042
koma2
ゲストわいひらさん、こんにちは。
数ヶ月前から Simplicity を便利に使わせていただいております。さて、私の環境では head cleaner プラグインを入れて
JavaScript を1ファイルに結合 & Minify しているのですが、
Simplicity の JavaScript が結合 & Minify の対象になってくれないので、
下記のような修正を行ないました。head cleaner なしでも動作に支障がないことは一応テストしましたので、
本体に取り入れていただけると助かります。以下、修正内容です。
1. wp_head() を呼び出す場所を、</head> 直前に変更。
修正の理由は、head cleaner が wp_head() の前の JavaScript だけを
処理対象にするからなのですが、
WordPress Codex によると “Put this template tag immediately before </head>”
とあるので、他のプラグイン等との兼ね合いからも
</head> 直前に移動するのが良いのかな? と思います。http://codex.wordpress.org/Function_Reference/wp_head
2. JavaScript の呼び出しに wp_enqueue_script() を利用
head cleanerの機能で JavaScript を </body> 直前に移動出来るのですが、
.jsファイル同士に依存関係がある場合(jQueryへの依存など)、
移動した時に正しい順番になってくれなくなるので
wp_enqueue_script() で依存関係を記述するようにしてみました。3. インラインスクリプトを javascript.js に移動
インラインスクリプトは依存関係を記述出来なくて、
jQuery に依存しているスクリプトの位置を動かした時に
エラーになるので、javascript.js に移動しました。
PHPからパラメータを読まないといけないところは
wp_localize_script() を使ってます。修正したファイルは、アップローダにアップしておきました。
Simplicity 1.4.0 20150103 からの修正です。
もし他の形式(patch等)でお送りした方がよろしければ教えてください。http://netakiri.net/uploader/src/up0068.zip
では、ご検討よろしくお願いいたします。
-
2015年1月23日 8:51 PM #18052
わいひら
キーマスターkoma2さんこんにちは。
コードの提供、ありがとうございます。
head cleanerでの不具合は、そういったことが原因だったんですね。
早速適用して試してみます。head cleaner なしでも動作に支障がないことは一応テストしましたので、
本体に取り入れていただけると助かります。こちらでも、動作確認はしますが、テーマカスタマイザーでの機能をいろいろ変更しても、大丈夫だったでしょうか?
以前、JavaScriptの順番を入れ替えて、不具合が出たので、順番入れ替えにちょっとした恐怖感があります;wp_localize_script()
こんな便利な関数があったんですね。勉強になります。
Simplicity 1.4.0 20150103 からの修正です。
もし他の形式(patch等)でお送りした方がよろしければ教えてください。1.4以降、これらのファイルに変更は加えていなかったはずなので、多分大丈夫だと思います。
-
2015年1月24日 3:44 AM #18078
koma2
ゲストわいひらさん、こんにちは。
こちらでも、動作確認はしますが、テーマカスタマイザーでの機能をいろいろ変更しても、大丈夫だったでしょうか?
以前、JavaScriptの順番を入れ替えて、不具合が出たので、順番入れ替えにちょっとした恐怖感があります;一応、影響が出そうな部分はオプションをいろいろ変更してテストしてみてます(head cleanerあり、なし含め)。ただ、オプションの数が多いし、組み合わせの問題もあるので、抜けがあるかもしれませんが…
-
2015年1月24日 6:05 AM #18099
わいひら
キーマスター一応、影響が出そうな部分はオプションをいろいろ変更してテストしてみてます(head cleanerあり、なし含め)。ただ、オプションの数が多いし、組み合わせの問題もあるので、抜けがあるかもしれませんが…
いえいえ、ありがとうございます。
どのような動作確認をされたのか、一応、確認しておきたかったもので。(僕1人での動作確認よりは、koma2さんもそれだけ確認していただいたと、聞くだけで安心感があるので)
全ての動作確認は設定数の多さからして無理だと思います。僕も昨日、パッチを当てて、影響がでそうなところを、思いつく限り、動作確認してみましたが、問題なく動作していました。もう既に、当サイトでもパッチが当たっている状態です。
重ねて近々、この変更のみでバージョンアップして、様子を見てみようと思います。 -
2015年1月25日 1:40 AM #18193
koma2
ゲストわいひらさん、こんばんは。
(僕1人での動作確認よりは、koma2さんもそれだけ確認していただいたと、聞くだけで安心感があるので)
そうですね。中々一人では不具合潰し切れないですからね…f(^_^;
僕も昨日、パッチを当てて、影響がでそうなところを、思いつく限り、動作確認してみましたが、問題なく動作していました。もう既に、当サイトでもパッチが当たっている状態です。
重ねて近々、この変更のみでバージョンアップして、様子を見てみようと思います。ありがとうございます。バージョンアップしたものが出たら、自分のサイトでも導入しようと思います。
よろしくお願いします。
-
2015年1月25日 12:49 PM #18235
わいひら
キーマスター昨日のSimplicity1.4.4で実装してみました。
よろしかったら、試していただけると幸いです。 -
2015年1月25日 1:57 PM #18243
koma2
ゲスト昨日のSimplicity1.4.4で実装してみました。
よろしかったら、試していただけると幸いです。ありがとうございます!早速試してみます。
-
-
投稿者投稿
- フォーラム「要望・機能追加など」には新規投稿および返信を追加できません。