В FireFox не работает input type=«radio» при добавлении его на страницу при помощи .append()

Question

[Евгений Елизаров]

Есть форма, там в зависимости от выбранного параметра показывается 3 радиобаттана или чекбокс, так вот выводимые радиобаттаны не работают в FF. В других браузерах проблем нет, а в FF удаётся выбрать только первый из 3 и всё. Код
html

 <select id="tracker" name="tracker" onchange="changeField()">
    <option></option>
    <option value="lostfilm.tv">lostfilm.tv</option>
    <option value="novafilm.tv">novafilm.tv</option>
</select>
...
<label id="changedField"></label>

//Меняем checkbox на radiobutton
function changeField()
{
	var tracker = document.getElementById("tracker").value;
	if (tracker == 'lostfilm.tv')
		$('#changedField').empty().append('<span class="quality"><input type="radio" name="hd" value="0"> SD качество<br/><input type="radio" name="hd" value="1"> HD качество<br/><input type="radio" name="hd" value="2"> HD MP4</span>');
	if (tracker == 'novafilm.tv')
		$('#changedField').empty().append('<input type="checkbox" name="hd"> HD качество</label>');
}

я в JS не силён, подскажите пожалуйста с чем косяк, а то люди жалуются :(
пс это для TorrentMonitor

Answer 1

[merlin-vrn]

jsfiddle.net/evK2H/ заменил ваш label на div — заработало в Fx. Ну label тут действительно был не очень уместен.
Ещё советую цеплять событие так, как сделано в фиддле, а не прописывать в onchange.

Comments

[Евгений Елизаров]

убрал lable и действительно заработало

Answer 2

[Николай Васильчук]

Если уж используете jQuery, используйте jQuery.
jsfiddle.net/Kh9Te/

Comments

[Александр Вульф]

Использовать switch/case не очень хорошая практика, имхо.

[Николай Васильчук]

Это чем же она вдруг стала плоха? Лучше вместо этого плодить if-ы?

[Александр Вульф]

switch(a) {
  case(1):
    console.log(1);
    break;
  case(2):
    console.log(2);
    break;
  default:
    console.log('not found');
    break;
}

Или:

if (a===1) {
  console.log(1);
} else if (a===2) {
  console.log(2);
} else {
  console.log('not found')
}

Второе намного логичнее, опять же имхо.

[Николай Васильчук]

switch(a) {
  case 1:
    console.log(1);
  case 2:
  case 3:
  case 4:
    console.log(1-4);
    break;
  case(5):
    console.log(5);
    break;
  default:
    console.log('not found');
    break;
}

Или:

if (a===1 || a===2 || a===3 || a===4) {
  if (a===1) {
    console.log(1);
  }
  console.log(1-4);
} else if (a===5) {
  console.log(5);
} else {
  console.log('not found')
}

Тут вам второе тоже больше нравится?

[Александр Вульф]

Да, вы правы. Я также сделал небольшой тест по скорости (http://jsfiddle.net/AmpCA/).
Всегда думал, что ifelse быстрее, но, видимо, ошибся.
Спасибо. Пересмотрю свое мнение о switchcase.

[Николай Васильчук]

Кстати, по скорости они примерно одинаковы (во всяком случае у меня в хроме)

[Александр Вульф]

И у меня.