要素から属性を取り除き、別のクリックで属性を復元するjQueryのクリック機能

Question

パレットに応じて色付けできる要素がSVGキャンバスにあります。特定の色がクリックされた場合は、塗りつぶしなしのオプションを追加しようとしています。他の色が選択されている場合は、その属性を削除します。

JSコード：

if(!$selectedElems){return;}                     

$selectedElems.each(function(){ 

$(this)[($(this).prop('tagName') === 'g' ? 'find' : 'filter')]('text[fill],rect[fill],circle,path').attr('fill',colour); 

var $noFillColour =  $('#mainToolFill div.toolPopup a.color15'); 
var $otherColour =  $('#mainToolFill div.toolPopup a:not("color15")'); 

if($($noFillColour).bind('click')) { 
    alert ($noFillColour); 
    $selectedElems.attr('fill-opacity', '0'); 
}else{ 
    ($($otherColour).bind('click')); 
    alert ($otherColour); 
    $selectedElems.removeAttr('fill-opacity'); 
} 

私は変数に問題があると思います。アラート内にObjectオブジェクトが表示されているためです。最初のイベントが発生しているので、選択した要素に塗りつぶし不透明度が適用されていますが、これはどの色が選択されたかに関係なく発生します。また、別の色をクリックすると、botはfill-opacity属性を削除します。

私はまた：notセレクタに何か問題があるかどうか疑問に思いました。私はまた試みた：

var $otherColour = $('#mainToolFill div.toolPopup a').not('#mainToolFill div.toolPopup a.color15'); 

誰かがこれにいくつかの光を投げることができれば感謝します。私はこれがこれを実装する最も経済的な方法ではないかもしれないことに注意してください。

もっとコードを表示する必要がある場合は、お知らせください。

Answer 1

は、私が働いている何かを持っていると思う：

これで唯一の問題は、それが高価であるかもしれないということである

$('#mainToolFill div.toolPopup a.color15').bind('click',function() { 
     $selectedElems.attr('fill-opacity','0'); 
}); 

$('#mainToolFill div.toolPopup a:not(".color15")').bind('click',function() { 
     $selectedElems.removeAttr('fill-opacity'); 
}); 

、塗りつぶし不透明度はcolor15が選択された後、しかし、他の色日常ストリップを追加しますfill-opacity属性が存在しない場合でも（colour15の場合を除いて）