2010-11-18 9 views
1

私は今日これについていくつかの助けを得ましたが、それを修正してそれを掃除する必要がありました。基本的にはタブ付きのインターフェースがあり、どのタブが表示されているかに基づいて適切なバナー広告を表示する必要があります。ブローカーとその他のグループは2つしかありません。したがって、ブローカのタブ(#BrokContent)が選択されている場合(ページの読み込み時と別のタブが表示されているときの両方をクリックした場合)、ブローカのバナーを表示します。他のすべての時間は他のバナーを示します。目に見えるタブに基づいてバナー広告を変更する

HTML - バナー広告

<div id="brokerAd"> 
    <h3 class="featured">Featured PARTNER</h3> 
    <div class="bannerTop">Im the broker banner 
    <!-- brokerAd End --></div> 
<!-- /bannerContain --></div> 

<div id="bannerContain"> 
    <h3 class="featured">Featured PARTNER</h3> 
    <div class="bannerTop">Other banners 
    <!-- AdSpeed.com End --></div> 
<!-- /bannerContain --></div> 

if ($("#BrokContent").is(":visible")) { 
      $("#brokerAd").show(); 
      $("#bannerContain").hide(); 
     } else { 
      $("#bannerContain").show(); 
      $("#brokerAd").hide(); 
     } 

     $("ul#flowtabs li a").click(function(){ 
      var bcVisible = $("#BrokContent").is(":visible"); 
      $("#brokerAd").toggle(bcVisible); 
      $("#bannerContain").toggle(!bcVisible); 
     }); 
+1

ここで何が間違っていますか?私はあなたのものを見逃しています。それはちょうどちょうどうまく働くので、これを読みます。 – jcolebrand

+0

@drachenstern - それはうまくいく、私はそれを簡素化する方法があるかどうか尋ねていた、私はコードをより簡潔にする方法を学ぶのが好きです。必要ではない/それが涼しい、ちょうど –

+0

@ drachensternを頼む - それを見ていただきありがとうございます、私はいつも、あなた自身のようなより多くの熟練したプログラマーの助けに感謝、再びありがとう! –

答えて

1

を簡素化する必要がJS私は簡単にそれを読んで、私はそれがより簡潔にする任意の方法を見つけることができません。私が考えることができるのは、clickハンドラの#brokContentでisvisibleのチェックを削除し、toggle()のままにしておくことです。それ以外は、私はそれについてだと思います。

関連する問題