Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Не считывает клик по кнопке, которая выводится через JS (https://javascript.ru/forum/misc/73326-ne-schityvaet-klik-po-knopke-kotoraya-vyvoditsya-cherez-js.html)

the_little 06.04.2018 15:43

Цитата:

Сообщение от j0hnik (Сообщение 482589)
у вас айдишкники одинаковые в каждом таком элементе, устраните проблему.

Имеете ввиду кнопку showtestform и форму form-comfort-1?
Да, это я оставил на тестовый период, пока выводилась только одна запись.
Сейчас сделал там тоже через i.

Но проблема сохраняется.

Клик по кнопке не срабатывает. Не то что не выводится форма. Даже в консоль не выводит сообщение, что кнопка нажата. И ошибок при этом нет.

the_little 06.04.2018 16:22

А как же тогда эту функцию переписать, чтобы менялся id?
$('#showtestform-0').click(function showtestform() {
  $('#form-comfort-0').fadeIn();
});


Там возможно через классы нужно тогда делать...
Делать .showtestform вместо id, находить ближайший div с классом, например form-container.
Тогда id-ники вообще там не нужны будут.

j0hnik 06.04.2018 16:24

клик соответственно тоже надо на каждую повесить ЦИКЛОМ, сделайте обновленный пример

j0hnik 06.04.2018 16:26

можно уже в имеющейся цикл на 13 строке, только после добавления всех потрохов

Manyasha 06.04.2018 16:39

Событие надо вешать либо, когда форма уже "нарисована" (строки 21-29 поставить перед 19-й), либо можно так написать:
$('#komfort-price').on('click', '#showtestform', function showtestform() {
  console.log('Открыть форму');
  $('#form-comfort-1').fadeIn();
});
 
$('#komfort-price').on('click', '.close-form',function closetestform() {
  console.log('Закрыть форму');
  $('#form-comfort-1').fadeOut();
});

про айдишки - не вникла, тут же одна форма, не вижу, где они повторяются...

j0hnik 06.04.2018 17:06

Manyasha,
не одна, вы почитайте полностью темку.

j0hnik 06.04.2018 17:08

хотя нормальной практикой будет сделать одну, а информацию добавлять по клику на кнопку.

Dilettante_Pro 06.04.2018 18:46

the_little,
Цитата:

Сообщение от the_little
Но там будет такая же функция для [i][2], [i][3], [i][4], а там есть нулевые.

А может лучше одну функцию для a[i][j], и проверять a[i][j] > 0 ?

the_little 06.04.2018 19:06

Цитата:

Сообщение от Dilettante_Pro (Сообщение 482614)
the_little,

А может лучше одну функцию для a[i][j], и проверять a[i][j] > 0 ?

даже не думал об этом.

там 4 кнопки, думал на каждую повесить свою функцию.

the_little 06.04.2018 19:10

Цитата:

Сообщение от j0hnik (Сообщение 482609)
хотя нормальной практикой будет сделать одну, а информацию добавлять по клику на кнопку.

сейчас подумаю как это сделать)
наверное так будет действительно правильно.

Если уж на то пошло, то формировать эти списки тоже можно не по клику, а при загрузке. а по клику - просто показывать этот список.

я просто не догоняю, если я сделал одну форму - круто. то как при нажатии на разные кнопки туда инфо добавлять?

так, есть мысли. сейчас все попробую переписать, чтобы одна форма была, и по клику только показывался блок.


Часовой пояс GMT +3, время: 22:50.