JavaScriptまわりの変更(head cleanerプラグインとの組み合わせ)

Simplicityの特徴 フォーラム 要望・機能追加など JavaScriptまわりの変更(head cleanerプラグインとの組み合わせ)

  • このトピックには6件の返信、2人の参加者があり、最後にkoma2により10年前に更新されました。
6件の返信スレッドを表示中
  • 投稿者
    投稿
    • #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

      では、ご検討よろしくお願いいたします。

    • #18052
      アバター画像わいひら
      キーマスター

      koma2さんこんにちは。
      コードの提供、ありがとうございます。
      head cleanerでの不具合は、そういったことが原因だったんですね。
      早速適用して試してみます。

      head cleaner なしでも動作に支障がないことは一応テストしましたので、
      本体に取り入れていただけると助かります。

      こちらでも、動作確認はしますが、テーマカスタマイザーでの機能をいろいろ変更しても、大丈夫だったでしょうか?
      以前、JavaScriptの順番を入れ替えて、不具合が出たので、順番入れ替えにちょっとした恐怖感があります;

      wp_localize_script()

      こんな便利な関数があったんですね。勉強になります。

      Simplicity 1.4.0 20150103 からの修正です。
      もし他の形式(patch等)でお送りした方がよろしければ教えてください。

      1.4以降、これらのファイルに変更は加えていなかったはずなので、多分大丈夫だと思います。

    • #18078
      koma2
      ゲスト

      わいひらさん、こんにちは。

      こちらでも、動作確認はしますが、テーマカスタマイザーでの機能をいろいろ変更しても、大丈夫だったでしょうか?
      以前、JavaScriptの順番を入れ替えて、不具合が出たので、順番入れ替えにちょっとした恐怖感があります;

      一応、影響が出そうな部分はオプションをいろいろ変更してテストしてみてます(head cleanerあり、なし含め)。ただ、オプションの数が多いし、組み合わせの問題もあるので、抜けがあるかもしれませんが…

    • #18099
      アバター画像わいひら
      キーマスター

      一応、影響が出そうな部分はオプションをいろいろ変更してテストしてみてます(head cleanerあり、なし含め)。ただ、オプションの数が多いし、組み合わせの問題もあるので、抜けがあるかもしれませんが…

      いえいえ、ありがとうございます。
      どのような動作確認をされたのか、一応、確認しておきたかったもので。(僕1人での動作確認よりは、koma2さんもそれだけ確認していただいたと、聞くだけで安心感があるので)
      全ての動作確認は設定数の多さからして無理だと思います。

      僕も昨日、パッチを当てて、影響がでそうなところを、思いつく限り、動作確認してみましたが、問題なく動作していました。もう既に、当サイトでもパッチが当たっている状態です。
      重ねて近々、この変更のみでバージョンアップして、様子を見てみようと思います。

    • #18193
      koma2
      ゲスト

      わいひらさん、こんばんは。

      (僕1人での動作確認よりは、koma2さんもそれだけ確認していただいたと、聞くだけで安心感があるので)

      そうですね。中々一人では不具合潰し切れないですからね…f(^_^;

      僕も昨日、パッチを当てて、影響がでそうなところを、思いつく限り、動作確認してみましたが、問題なく動作していました。もう既に、当サイトでもパッチが当たっている状態です。
      重ねて近々、この変更のみでバージョンアップして、様子を見てみようと思います。

      ありがとうございます。バージョンアップしたものが出たら、自分のサイトでも導入しようと思います。

      よろしくお願いします。

    • #18235
      アバター画像わいひら
      キーマスター

      昨日のSimplicity1.4.4で実装してみました。
      よろしかったら、試していただけると幸いです。

    • #18243
      koma2
      ゲスト

      昨日のSimplicity1.4.4で実装してみました。
      よろしかったら、試していただけると幸いです。

      ありがとうございます!早速試してみます。

6件の返信スレッドを表示中
  • フォーラム「要望・機能追加など」には新規投稿および返信を追加できません。
スポンサーリンク
アドセンス(大)
アドセンス(大)