Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #11 (permalink)  
Старый 06.04.2018, 15:43
Профессор
Отправить личное сообщение для the_little Посмотреть профиль Найти все сообщения от the_little
 
Регистрация: 25.09.2017
Сообщений: 186

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

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

Клик по кнопке не срабатывает. Не то что не выводится форма. Даже в консоль не выводит сообщение, что кнопка нажата. И ошибок при этом нет.
Ответить с цитированием
  #12 (permalink)  
Старый 06.04.2018, 16:22
Профессор
Отправить личное сообщение для the_little Посмотреть профиль Найти все сообщения от the_little
 
Регистрация: 25.09.2017
Сообщений: 186

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


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

Последний раз редактировалось the_little, 06.04.2018 в 16:29.
Ответить с цитированием
  #13 (permalink)  
Старый 06.04.2018, 16:24
Аватар для j0hnik
Профессор
Отправить личное сообщение для j0hnik Посмотреть профиль Найти все сообщения от j0hnik
 
Регистрация: 01.12.2016
Сообщений: 3,650

клик соответственно тоже надо на каждую повесить ЦИКЛОМ, сделайте обновленный пример
Ответить с цитированием
  #14 (permalink)  
Старый 06.04.2018, 16:26
Аватар для j0hnik
Профессор
Отправить личное сообщение для j0hnik Посмотреть профиль Найти все сообщения от j0hnik
 
Регистрация: 01.12.2016
Сообщений: 3,650

можно уже в имеющейся цикл на 13 строке, только после добавления всех потрохов
Ответить с цитированием
  #15 (permalink)  
Старый 06.04.2018, 16:39
Профессор
Отправить личное сообщение для Manyasha Посмотреть профиль Найти все сообщения от Manyasha
 
Регистрация: 21.09.2015
Сообщений: 196

Событие надо вешать либо, когда форма уже "нарисована" (строки 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();
});

про айдишки - не вникла, тут же одна форма, не вижу, где они повторяются...
Ответить с цитированием
  #16 (permalink)  
Старый 06.04.2018, 17:06
Аватар для j0hnik
Профессор
Отправить личное сообщение для j0hnik Посмотреть профиль Найти все сообщения от j0hnik
 
Регистрация: 01.12.2016
Сообщений: 3,650

Manyasha,
не одна, вы почитайте полностью темку.
Ответить с цитированием
  #17 (permalink)  
Старый 06.04.2018, 17:08
Аватар для j0hnik
Профессор
Отправить личное сообщение для j0hnik Посмотреть профиль Найти все сообщения от j0hnik
 
Регистрация: 01.12.2016
Сообщений: 3,650

хотя нормальной практикой будет сделать одну, а информацию добавлять по клику на кнопку.
Ответить с цитированием
  #18 (permalink)  
Старый 06.04.2018, 18:46
Профессор
Отправить личное сообщение для Dilettante_Pro Посмотреть профиль Найти все сообщения от Dilettante_Pro
 
Регистрация: 27.11.2015
Сообщений: 2,899

the_little,
Сообщение от the_little
Но там будет такая же функция для [i][2], [i][3], [i][4], а там есть нулевые.
А может лучше одну функцию для a[i][j], и проверять a[i][j] > 0 ?
Ответить с цитированием
  #19 (permalink)  
Старый 06.04.2018, 19:06
Профессор
Отправить личное сообщение для the_little Посмотреть профиль Найти все сообщения от the_little
 
Регистрация: 25.09.2017
Сообщений: 186

Сообщение от Dilettante_Pro Посмотреть сообщение
the_little,

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

там 4 кнопки, думал на каждую повесить свою функцию.
Ответить с цитированием
  #20 (permalink)  
Старый 06.04.2018, 19:10
Профессор
Отправить личное сообщение для the_little Посмотреть профиль Найти все сообщения от the_little
 
Регистрация: 25.09.2017
Сообщений: 186

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

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

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

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



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Help подгрузка js через onclick sarex jQuery 3 27.04.2016 11:13
При подгрузке страницы через AJAX JS не отрабатывается, но подгружается vityurov Общие вопросы Javascript 2 20.06.2013 12:24
Динамическое создание iframe в Firefox mrbean11 Firefox/Mozilla 8 02.11.2012 21:23
AJAX не работает с формой, которая выводится через js wayzer Элементы интерфейса 2 15.09.2012 21:01
Повтор музыки в JS через flash kuper Общие вопросы Javascript 0 13.12.2011 10:57