divの枠線を表示または非表示にする - Javascript?

私はカスタムスタンプを作成するマルチパートフォームを持っています。私はそれのメインchunckはうまく動作しているが、境界を同じ色のテキストの色の選択を取るように戦っています。私はボーダーpxのドロップダウンリストとフォントの色のドロップダウンリストを持っています。フォントの色が動作し、私は仕事をするための境界線と、フォントの色と同じ色をとる必要があります。 ここで間違ったことを私に教えてもらえますか?

http://jsfiddle.net/ShauniD/ELER2/11/ (this is the full code, so you can see what i am trying to achieve.

私はこれを行うためのより良い方法があることを知っていますが、私は非常に新しいです、そして、小片のための作品を学んでいます。もしそうなら簡単に行けます。

Javascriptは次のとおりです:

function setColor()   {     
var color = document.getElementById("color").value;     
document.getElementById("myDiv").style.color = color;   
} 
function addContent(divName, content) {
 document.getElementById(divName).innerHTML = content;
}
 function border(border)    {
document.getElementById("myDiv").style.borderWidth = border;
}

HTMLは次のとおりです。

    <tr>

        <td>
            <input name="myContent1">
            <input type="button" value="Add content" onClick="addContent('lineTwo', document.myForm.myContent1.value); setCookie('content', document.myForm.myContent1.value, 7);">
        </td>
</tr>
<tr><td>
 </td></tr> 
    <tr><td>
    </td></tr>
</form>

<div id="myDiv">
        <div id="lineOne"></div>
        <div id="lineTwo"></div>
        <div id="lineThree"></div>
        <div id="lineFour"></div>
    </div>

それが何であるかを理解するために、完全なコードを見てください。

2

2 答え

あなたの関数の名前を 'changeborder'に変更してください。それはあなたのために機能します。

ブラウザは、要素と同じ名前を使用することに同意しません。

2
追加された
そして、なぜブラウザがここに同意したのですか?
追加された 著者 Oleg Mikheev,
または、より簡単に、属性 id = "border" を削除または変更します。同じ識別子を使用することがなぜここで問題になるのかは明らかではありません。
追加された 著者 Jukka K. Korpela,
あなたの例で変更を保存します。私はそこを見ます
追加された 著者 Hadas,
onchangeイベントで関数の名前を変更することは忘れてしまいます。また、関数 'borderWidth'を 'border'に変更すると、
追加された 著者 Hadas,
@Olegありがとう!私はすでにそれを見てきました(私はあなたに答えてみたいです...)
追加された 著者 Hadas,
あなたが提案したように関数名を変更し、attribut(id = "border")を削除しました。私はまだそれを働かせるように思わない。
追加された 著者 Shauni Daniels,
更新されたバージョンとリンクは以下のとおりです。 jsfiddle.net/ShauniD/ELER2
追加された 著者 Shauni Daniels,
おかげでミル。 。 。愚かな間違いは大きな問題につながる。 。 。その初心者のことを推測する。どうもありがとうございました。
追加された 著者 Shauni Daniels,
@Olegこの素晴らしい説明をありがとう。それは学習に役立ちますし、コーディングを修正するための正しい道を私に教えてくれます。
追加された 著者 Shauni Daniels,

あなたのオプションの値は "0px none"でなければなりません。おそらく、 "none"または "0px"で十分でしょう。

0
追加された
なぜ-1?それは動作します。さて、関数名で間違いはありませんでした。
追加された 著者 reporter,
JavaScript - 日本のコミュニティ
JavaScript - 日本のコミュニティ
2 参加者の

日本人コミュニティのjavascript