3

みなさん、こんにちは。

<figure>とを使用して、通常の画像タグからHTML5画像キャプションを作成するスクリプトを作成しました<figcaption>。

私のCMSはFCKEditorを使用しており、常に段落内に埋め込み画像を配置します。したがって、私のスクリプトは<figcaption>画像の周りを構築し、それを段落の外に移動します（html5を参照、段落内のfigure / figcaptionは予測できない出力を提供します））。

私が書いたスクリプトは機能しますが、DOMを1回だけトラバースする方法がわからなかったため、DOMを2回トラバースします。JQueryに精通している誰かが、スクリプトを単純化/改善する方法についていくつかの提案を提供していただければ幸いです。

ありがとう、-NorthK

// use like this: 
// <img class="caption" src="http://placehold.it/350x150" alt="Sample image caption" />
//
$(document).ready(function() {
// iterate over each element with img.caption
$('img.caption').each(function() {
    var classList = $(this).attr('class');  // grab the image's list of classes, if any
    $(this).wrap('<figure class="' + classList + '"></figure>'); // wrap the <img> with <figure> and add the saved classes
    $(this).after('<figcaption>' + $(this).attr('alt') + '</figcaption>'); // add the caption
    $(this).removeAttr('class'); // remove the classes from the original <img> element
});

// now iterate over each figure.caption we built, and relocate it to before its closest preceding paragraph
$('figure.caption').each(function() {
    $(this).parent('p').before($(this));
});
})
4

2 に答える 2

5

各要素をラップするときに、ラッパーを配列に保存してから、DOMを再トラバースするのではなく、配列を処理できます。

$(document).ready(function() {
    var wrappers = [];

    // iterate over each element with img.caption
    $('img.caption').each(function() {
        var $this = $(this);
        var classList = this.className;  // grab the image's list of classes, if any
        $this.wrap('<figure class="' + classList + '"></figure>'); // wrap the <img> with <figure> and add the saved classes

        // Remember the wrapper here:
        wrappers.add($this.parent());

        $this.after('<figcaption>' + $(this).attr('alt') + '</figcaption>'); // add the caption
        $this.removeAttr('class'); // remove the classes from the original <img> element
    });

    // now iterate over each figure.caption we built, and relocate it to before its closest preceding paragraph
    $.each(wrappers, function(index, item) {
        $(item).parent('p').before(item);
    });

    // And if you're done with it, release it:
    wrappers = null;
});

簡単な例を次に示します。

HTML：

<p><img class='caption' src='http://www.gravatar.com/avatar/6d8ebb117e8d83d74ea95fbdd0f87e13?s=48&d=identicon&r=PG'></p>
<p><img class='caption' src='http://www.gravatar.com/avatar/ca3e484c121268e4c8302616b2395eb9?s=48&d=identicon&r=PG'</p>

JavaScript：

jQuery(function($) {

  var wrappers = [];
  $("img.caption").each(function() {
    var $this = $(this);
    $this.wrap("<figure class='" + this.className + "'>");
    wrappers.push($this.parent('figure'));
  });
  $.each(wrappers, function(index, item) {
    $(item).addClass("foo");
  });
  wrappers = null;
});

ライブコピー

オフトピック：あなたは効率に興味があるようですので、私は言及します：へのすべての呼び出しに$(this)は、複数の関数呼び出しとメモリ割り当てが必要です。繰り返し実行するのではなく、ループごとに1回実行して、結果をキャッシュします。例として、上記でそれを行いました。常に同じ関数で書き込むこと$(this)は、パフォーマンスの観点からは理想的ではありませんが、99％の場合、それは問題ではありません。あなたがたくさんの要素を扱っているなら、それはそうします。

于 2011-03-25T22:47:02.613 に答える
0

このメソッドを使用して、最初の反復でunwrapを取り除くことができます。<p>連鎖を使用して構文を単純化することもできます。

$('.caption').each(function() {
    $(this)
        .unwrap() // ditch the conflicting pararaph parent wrapper
        .wrap('<figure class="' + $(this).attr('class') + '" />') // wrap the figure tag instead, jQuery can handle a self closing tag
        .after('<figcaption>' + $(this).attr('alt') + '</figcaption>') // add the caption
        .removeAttr('class'); // remove the classes from the original <img> element
});

http://jsfiddle.net/pxfunc/AGgL2/

于 2011-03-25T22:45:15.870 に答える