laimas,
Простите, если я не прав и туплю:) Но ведь это родитель основного флажка и блока со второстепенными. на нём висит ваш код и код показать/скрыть дочерний блок .upsale__block__bot-right__sub-wrap, который выступает обёрткой для блоков со второстепенными чекбоксами. Т.е. обработчик чекбокса там только один - ваш, а прерывание висит, чтобы при нажатии на чекбокс не срабатывал скрипт показать/скрыть блок |
Цитата:
|
laimas,
Я наверное тупой :cray: НО загвоздка не в главном чекбоксе, а то что событие клика не назначено на родителя второстепенных чекбоксов, чтобы активировать дочерний чекбокс. Схематично если, то сейчас: блок 1 > input. click происходит по умолчанию на input, чтобы галка встала. А надо: блок 1 > input click должен быть на блок 1, чтобы галка встала в input. Представим что input в состоянии disabled и меняет своё состояние при нажатии на (в данном случае) родителя. Но при этом должен работать и тот код, который влияет на главный чекбокс. |
Цитата:
Цитата:
|
laimas,
.upsale__block__sub-right-wrap - родитель для всех блоков и на него вешается событие клика (show/hide блок) для .upsale__block__bot-right__sub-wrap второстепенные input должны срабатывать при клике на .floor__item-wrap. Выше не надо подниматься для события click по этому чекбоксу Получается логика такая: кликаем на .floor__item-wrap и дочерний для него чекбокс получает/снимает с себя галку, а далее уже работает функция установки и проверки состояния on/off для главного чекбокса. Чтобы не путаться, то можно сделать второстепенные чекбоксы как disabled (в html хардкодом) и, тогда нужно обработчик клика по чекбоксу перенести на .floor__item-wrap. Вот именно этот перенос обработчика клика и нужно добавить в ваш код, чтобы всё работало не конфликтуя:) Порядок (при клике на .floor__item-wrap): click по .floor__item-wrap -> проверка состояния дочернего input для этого блока -> ставим/снимаем галку в input (if есть галка уже/нет галки) -> проверка на состояние остальных input в оставшихся .floor__item-wrap -> снятие/установка галки в inpit в .upsale__block__bot-right__title-wrap <div class="upsale__block__sub-right-wrap"> <div class="upsale__block__bot-right__title-wrap dflex"> <div class="upsale__block__bot-right__title dflex"> <input id="floor" type="checkbox" class="block__check" onclick="event.stopPropagation()"> // этот input "главный". stopPropagation() добавлен, чтобы при нажатии на input не срабатывал скрипт click для upsale__block__bot-right__title-wrap <span> <img class="arrow arrow-up" src="" alt="arrow_down"> Choose your floor </span> </div> <div class="col-title__price-wrap dflex"> <div></div> <div></div> <div></div> </div> </div> <div class="upsale__block__bot-right__sub-wrap dnone" style="display: block;"> <div class="montage__content-wrap"> <span class="montage__text">intro text <div class="montage__floor-wrap montage__comp-wrap dflex"> <div class="montage__floor dflex"> <div class="floor__item-wrap"> //на этот блок должен работать click и включать/выключать дочерний второстепенный input <div class="floor__item-inner"> <div class="floor__item"> <input id="first_floor" type="checkbox" name="groundfloor" value="groundfloor"> //второстепенный input <div class="floor__item__img"> <img src="" alt=""> </div> <span class="floor__item__text">Ground floor</span> </div> </div> </div> //закрытие блока .floor__item-wrap <div class="floor__item-wrap"> //на этот блок должен работать click и включать/выключать дочерний второстепенный input <div class="floor__item-inner"> <div class="floor__item"> <input id="sec_floor" type="checkbox" name="middlefloor" value="middlefloor"> //второстепенный input <div class="floor__item__img"> <img src="" alt=""> </div> <span class="floor__item__text">Second floor</span> </div> </div> </div> //закрытие блока .floor__item-wrap <div class="floor__item-wrap"> //на этот блок должен работать click и включать/выключать дочерний второстепенный input <div class="floor__item-inner"> <div class="floor__item"> <input id="third_floor" type="checkbox" name="topfloor" value="topfloor"> //второстепенный input <div class="floor__item__img"> <img src="" alt=""> </div> <span class="floor__item__text">Third floor</span> </div> </div> </div> //закрытие блока .floor__item-wrap <div class="floor__item-wrap"> //на этот блок должен работать click и включать/выключать дочерний второстепенный input <div class="floor__item-inner"> <div class="floor__item"> <input id="all_floor" type="checkbox" name="apartment" value="apartment"> //второстепенный input <div class="floor__item__img"> <img src="" alt=""> </div> <span class="floor__item__text">Apartment Building</span> </div> </div> </div> //закрытие блока .floor__item-wrap </div> </div> </div> </div> </div> |
Охренеть. Чем в конце концов должен заниматься флажок id="floor":
а) только показывать/скрывать блок? б) показывать/скрывать блок и выбирать все второстепенные флажки? |
laimas,
Цитата:
|
Цитата:
Блоков upsale__block__sub-right-wrap на этой странице всегда два? |
Цитата:
Цитата:
https://prnt.sc/pk5amz |
Ясно.Дело в том, что в первом блоке оказывается есть и другие поля, но типа file, а значит селектор для делегирования нельзя так указывать.
|
laimas,
первый блок не смотрите. Надо на примере второго только создать скрипт. первый блок законченный уже. он такой один будет и его скрипты отдельные. Кроме скрипта show/hide для показа дочернего .upsale__block__bot-right__sub-wrap. Но и тот работает по принципу this.children() |
Цитата:
Код этого обработчика $('.upsale__block__sub-right-wrap').on('click', '.upsale__block__bot-right__title-wrap', function() ... заменить на: $('div.upsale__block__bot-right__title-wrap').click(function() { $(this).next() .slideToggle(100) .end() .find('img').toggleClass('arrow_down arrow-up') .end() .find('.montage__more').hide() .end() .find('.invite').show(); }); Здесь делегирование совсем не требуется. Правда зачем в нем строки: $(this).siblings('.upsale__block__bot-right__sub-wrap').find('.montage__more').hide(); $(this).siblings('.upsale__block__bot-right__sub-wrap').find('.invite').show(); (в новом коде это четыре последних операции в цепочке) если можно сразу сделать нужно видимым/скрытым. Код обработки изменения флажков заменить на этот: $('div.upsale__block__sub-right-wrap').slice(1).on('change', 'input', function(e) { var a = $(e.delegateTarget).find('input'), chk = a.slice(1); if(a.eq(0)[0]==this) { chk.prop('checked', false); this.disabled = true; }else { var c = chk.filter(':checked').length; a.eq(0).prop({checked: c, disabled: !c}); } }); Он устанавливается на блоки кроме первого. |
Вложений: 1
laimas,
Цитата:
- при нажатии на .btn_more (желтая кнопка) скрывается кнопка (синяя) .invite и открывается появляется блок .montage__more. Когда повторно нажимаем на .btn_more, то montage__more скрывается, а кнокпа .invite (синяя) появляется на том месте где была скрыта. Причем кнопка .invite дублируется в блоке .montage__more, когда тот открывается, НО она не участвует в срипте show/hide (и не должна). - если развернут блок .montage__more, то при нажатии на .upsale__block__bot-right__title-wrap этот блок должен свернуться и кнопка .invite должна стать show(), чтобы при последующем открытии она была доступна пользователю, т.к. если этого не сделать, то она не появится. на картинках всё показал. Надеюсь понятно объясняю, а не только для меня мои мысли понятны:) Но опять же это не решает проблему нажатия на .floor__item-wrap, чтобы сработал его дочерний чекбокс |
Цитата:
Цитата:
|
Цитата:
|
Цитата:
наверное так и сделаю. А то 2 дня уже страдаем, а до этого еще я сам 2 потратил. laimas, Спасибо за Ваше терпение и помощь! Жаль нельзя плюсиков больше поставить - я вам вчера уже один поставил:) |
Цитата:
$('div.upsale__block__sub-right-wrap').slice(1).on('change', 'input', function(e) { var chk = $(e.delegateTarget).find('input'); if(chk.eq(0)[0]==this) { chk.slice(1).prop('checked', false); this.disabled = true; }else { var c = chk.slice(1).filter(':checked').length; chk.eq(0).prop({checked: c, disabled: !c}); } }).find('.floor__item').each(function() { $(this).children().wrapAll('<label/>') }); |
Заменил div на label и теперь новая задача возникает:
Раньше, когда галка ставилась для второстепенного input, то родительскому блоку добавлялся background цветом, а галка снималась - background белым делался. Как теперь это добавить в тот код, что имеется?:) $('div.upsale__block__sub-right-wrap').slice(1).on('change', 'input', function(e) { var a = $(e.delegateTarget).find('input'), chk = a.slice(1); if(a.eq(0)[0]==this) { chk.prop('checked', false); this.disabled = true; }else { var c = chk.filter(':checked').length; a.eq(0).prop({checked: c, disabled: !c}); } }); |
Цитата:
|
Цитата:
А вот про цвет не писал - думал сам справлюсь, но не понимаю как связать это с вашим кодом - не работает то, что я пробую. |
laimas,
Цитата:
|
Цитата:
.floor__item.active { background-color: #A5D8E4; } $('div.upsale__block__sub-right-wrap').slice(1).on('change', 'input', function(e) { var a = $(e.delegateTarget).find('input'), chk = a.slice(1); if(a.eq(0)[0]==this) { chk.prop('checked', false); this.disabled = true; $(e.delegateTarget).find('.floor__item').removeClass('active') }else { var c = chk.filter(':checked').length; a.eq(0).prop({checked: c, disabled: !c}); } $(this).closest('.floor__item').toggleClass('active') }).find('.floor__item').each(function() { $(this).children().wrapAll('<label/>') }); |
Кстати, код свернуть/развернуть надо тоже дополнить:
$('div.upsale__block__bot-right__title-wrap').click(function(e) { if(e.target.localName!='input') { $(this).next() .slideToggle(100) .end() .find('img').toggleClass('arrow_down arrow-up') .end() .find('.montage__more').hide() .end() .find('.invite').show(); } }); Иначе по флажку тоже будет сворачивать. |
Цитата:
И тут есть момент - closest() не поддерживается IE даже 10+, и при снятии главной галочки, второстепенные снимаются, а окраска с их блоков нет. Подключать 2 полифила для closest() не очень хочется, поэтому я и решил переписать с JS на jQuery код, потому что компактней выходит:) P.S. окрашивать надо не .floor__item, а .floor__item-wrap |
Цитата:
Цитата:
$('div.upsale__block__sub-right-wrap').slice(1).on('change', 'input', function(e) { var a = $(e.delegateTarget).find('input'), chk = a.slice(1); if(a.eq(0)[0]==this) { chk.prop('checked', false); this.disabled = true; $(e.delegateTarget).find('.floor__item-wrap').removeClass('active') }else { var c = chk.filter(':checked').length; a.eq(0).prop({checked: c, disabled: !c}); } //$(this).closest('.floor__item-wrap').toggleClass('active') //а так для инвалидов $(this).parents('.floor__item-wrap').toggleClass('active') }).find('.floor__item-wrap > div').wrap('<label/>'); |
laimas,
Вы мой кумир:) Спасибо огромное! Это то что надо!:dance: Правда IE всё равно не хочет работать с нажатием на блок, а не на input. Но зато другие работают - этого достаточно. Тему скрипта можно закрывать, а тему благодарностей laimas открывать:thanks: p.s. Цитата:
|
Цитата:
Aruta, два дня не копался, наверное надо давать понятные имена, а то можно легко запутаться с вашим кодом и всё не понимать, что такое там написано! Вам такое почему-то запрещено писать? Я имею в виду про то, что вы печатаете много <div>, и ещё обёртки и ещё обёртки... <details class="upsale_sub_right"> <summary> <input type="checkbox"> Choose your floor </summary> <p class="montage_text">intro text</p> <div class="floor_items"> <label> <input type="checkbox" name="groundfloor"> <span class="floor_item_text">Ground floor</span> </label> <label> <input type="checkbox" name="middlefloor"> <span class="floor_item_text">Second floor</span> </label> <label> <input type="checkbox" name="topfloor"> <span class="floor_item_text">Third floor</span> </label> <label> <input type="checkbox" name="apartment"> <span class="floor_item_text">Apartment Building</span> </label> </div> </details> <script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.4.1/jquery.min.js"></script> <script> $(".upsale_sub_right").on("change", function(event) { var inputs = $(this).find("input:not(summary input)"); var parentInput = $(this).find("summary input"); if(parentInput.is(event.target)) { inputs.prop("checked", parentInput.prop("checked")); } else { var checkedInputs = inputs.filter(":checked"); parentInput.prop({ checked: checkedInputs.length !== 0, indeterminate: checkedInputs.length !== 0 && inputs.length !== checkedInputs.length }); } $(this) .find(".floor_items > label.active:has(:not(:checked))").removeClass("active") .end() .find(".floor_items > label:has(:checked)").addClass("active") ; }); </script> <style> .floor_items > label { display: block; } .floor_items > .active { background-color: #A5D8E4; } </style> |
Хотел сделать финальный вариант, чтобы другие сразу могли его увидеть, но он не работает тут как надо - проще файлами прикладывать:)
|
Цитата:
|
laimas,
этот сайт еще даже не запущен:( Желание заказчика, чтобы работало везде более менее одинаково - вот и кручусь. Сайт мне достался в том виде, в котором есть сейчас. Моих доработок там только пара страниц всего, включая backend для них. Вот теперь могу спокойно приниматься за back для этой страницы. А по поводу поддержки closest(), я знаю что в JS closest и IE не дружат. И тут я думал тоже такая же зависимость причина-следствие сохраняется:) Спасибо за пояснение! |
Цитата:
|
laimas,
Теперь вы путаете:) О jQuery не говорю, только о чистом JS. https://caniuse.com/#search=closest |
Цитата:
$(this).closest('.floor__item-wrap').toggleClass('active') |
laimas,
Цитата:
Короче, не задумывайтесь об этом:) не стоит оно того - только время в пустую. |
Часовой пояс GMT +3, время: 07:35. |