Onclick для всех картин

@khrustal · Регистрация: 26.11.2013

Author24 — интернет-сервис помощи студентам

Можно ли применить

Javascript

1	onclick = 'openImageWindow(this.src);'

ко всем картинам внутри конкретного div-а?

@mrtoxas · 20.08.2016, 13:31

HTML5

<div class="wrapper">
  <img class="imgGetSize" src="" alt="" />
  <img class="imgGetSize" src="" alt="" /> 
</div>

Javascript

var images = document.querySelectorAll('.wrapper > img');
images.forEach(function(elem){
  elem.onclick= function openImageWindow() {
    var image = new Image();     
    var width = this.width;
    var height = this.height;
    window.open(this.src,"Image","width=" + width + ",height=" + height);
  }
});

@khrustal · 21.08.2016, 10:48 **[ТС]**

mrtoxas, дело в том, что в тэги <img> вписать ничего не могу. Единственное, могу весь текст с <img> -ами разместить в блок. Т.е., JS должен обрабатывать все картины в wrapper-e, а class="imgGetSize" не должно быть.

@mrtoxas · 21.08.2016, 10:52

Убирайте класс из тэгов. Это я экспериментировал и забыл удалить. Выборка идет по .wrapper > img

@khrustal · 21.08.2016, 12:44 **[ТС]**

mrtoxas, да, я уже догадалась. Но не работает.
Думаю дело в том, что на самом деле в wrapper могут быть другие div-ы, и в них только картины.
wrapper > img означает прямая внедренность?

@mrtoxas · 21.08.2016, 12:55

Это означает непосредственный потомок. Попробуйте без ">". Я ж не знаю структуру документа

@khrustal · 21.08.2016, 12:58 **[ТС]**

mrtoxas, тоже нет
а .wrapper > div> img
должен ведь работать... а не работает

@mrtoxas · 21.08.2016, 13:05

Дайте структуру html

Добавлено через 5 минут
querySelectorAll('.wrapper img') зацепит абсолютно все <img>, которые вложены в .wrapper, вне зависимости от их дальнейших вложенностей.

@khrustal · 21.08.2016, 13:08 **[ТС]**

PHP/HTML

<html>
<head>....</head>
<body>
....
<div class="secdes">
.....текст статьи....
</div>
 
<script type="text/javascript">
var images = document.querySelectorAll('.secdes > img');
images.forEach(function(elem){
  elem.onclick= function openImageWindow() {
    var image = new Image();     
    var width = this.width;
    var height = this.height;
    window.open(this.src,"Image","width=" + width + ",height=" + height);
  }
});
</script>
 
</body>

.....текст статьи.... это текст созданный в wysiwyg редакторе, т.е. там может быть все что угодно. И для всех картин там нужно применить этот скрипт.

@mrtoxas · 21.08.2016, 13:14

HTML5

<div class="wrapper">
  <p>Tекст стати</p>
  <img src="#" alt="" />
  <div class="secdes">
    <p>Tекст стати</p>
    <img src="#" alt="" />
    <div class="secdes_second">
      <p>Tекст стати</p>
      <img src="#" alt="" />
    </div>
  </div>
</div>

Javascript

var images = document.querySelectorAll('.wrapper img');
images.forEach(function(elem){
  elem.onclick= function openImageWindow() {
    var image = new Image();     
    var width = this.width;
    var height = this.height;
    window.open(this.src,"Image","width=" + width + ",height=" + height);
  }
});

Результат

@khrustal · 21.08.2016, 13:24 **[ТС]**

mrtoxas, оказывается не работает в firefox, а в гуглхром нормально. Не знаете, отчего это?
Кроме того, картины в новом окне не показываются оригинальным размером, а надо так.

@mrtoxas · 21.08.2016, 13:49

почему-то forEach не срабатывает. Попробуйте циклом, правда не уверен в правильности решения, может кто поправит:

Javascript

var images = document.querySelectorAll('.wrapper img');
  for (i = 0; i < images.length; ++i) {  
  images[i].onclick= function openImageWindow() {    
    var image = new Image();     
    var width = this.width;
    var height = this.height;
    window.open(this.src,"Image","width=" + width + ",height=" + height);
  }
}

Добавлено через 13 минут
Вот, так лучше:

Javascript

var images = document.querySelectorAll('.wrapper img');
[].forEach.call(images, function(elem) {
   elem.addEventListener("click", function(){ 
    var image = new Image();     
    var width = this.width;
    var height = this.height;
    window.open(this.src,"Image","width=" + width + ",height=" + height);
  });
});

@khrustal · 21.08.2016, 14:08 **[ТС]**

Спасибо большое, сработало.
Еще если можно немного доделать. Можно ли сделать так, чтобы это новое окно
1. имело размер картины (теперь оно имеет размер картины, которое указано в html коде, а там картина показывается в уменьшенном размере)
2. окно не имело строки, где указан URL
3. окно закрывалось при клике вне пределах окна

@mrtoxas · 21.08.2016, 15:38

Я думаю, такое надо делать не через открытие окна, а при помощи css. Через JS только подгружать картинки в блок и задавать ему размеры.

Добавлено через 1 час 17 минут
Надо допиливать, и придумать как загружать полную картинку, но смысл такой:

HTML5

<div class="wrapper">
  <p>Tекст стати</p>
   <img src="#" alt="" />
  <div class="secdes">
    <p>Tекст стати</p> 
    <img src="#" alt="" />
  </div> 
</div>

Javascript

var images = document.querySelectorAll('.wrapper img');
var poPupDiv = document.createElement('div');
poPupDiv.className = "pop-up";
poPupDiv.style.width = '100%';
poPupDiv.style.height = '100%';
poPupDiv.style.top = '0';
poPupDiv.style.left = '0';
poPupDiv.style.display = 'none';
poPupDiv.style.position = 'fixed';
poPupDiv.style.background = 'rgba(0,0,0,.8)';
document.body.appendChild(poPupDiv);
var poPup = document.querySelector('.pop-up');
poPup.onclick = function() {
      this.style.display='none';
    }
images.forEach(function(elem) {
    elem.onclick = function openImageWindow() {
    poPupDiv.style.display = 'block';
    imgHeight = getComputedStyle(elem).height;
    imgWidth = getComputedStyle(elem).width;
    poPup.innerHTML = '<img src="' + elem.src + '" alt="" \
        style="position:fixed;top:50%;left:50%;\
        transform:translate(-50%,-50%);background:#fff";/>';        
  }
});

Результат

@khrustal 2 / 2 / 0 Регистрация: 26.11.2013 Сообщений: 248
	21.08.2016, 10:48 [ТС]	3
	mrtoxas, дело в том, что в тэги <img> вписать ничего не могу. Единственное, могу весь текст с <img> -ами разместить в блок. Т.е., JS должен обрабатывать все картины в wrapper-e, а class="imgGetSize" не должно быть. 0

@mrtoxas 3825 / 2675 / 1521 Регистрация: 12.07.2015 Сообщений: 6,672 Записей в блоге: 4
	21.08.2016, 10:52	4
	Убирайте класс из тэгов. Это я экспериментировал и забыл удалить. Выборка идет по .wrapper > img 0

@khrustal 2 / 2 / 0 Регистрация: 26.11.2013 Сообщений: 248
	21.08.2016, 12:44 [ТС]	5
	mrtoxas, да, я уже догадалась. Но не работает. Думаю дело в том, что на самом деле в wrapper могут быть другие div-ы, и в них только картины. wrapper > img означает прямая внедренность? 0

@mrtoxas 3825 / 2675 / 1521 Регистрация: 12.07.2015 Сообщений: 6,672 Записей в блоге: 4
	21.08.2016, 12:55	6
	Это означает непосредственный потомок. Попробуйте без ">". Я ж не знаю структуру документа 0

@khrustal 2 / 2 / 0 Регистрация: 26.11.2013 Сообщений: 248
	21.08.2016, 12:58 [ТС]	7
	mrtoxas, тоже нет а .wrapper > div> img должен ведь работать... а не работает 0

@mrtoxas 3825 / 2675 / 1521 Регистрация: 12.07.2015 Сообщений: 6,672 Записей в блоге: 4
	21.08.2016, 13:05	8
	Дайте структуру html Добавлено через 5 минут querySelectorAll('.wrapper img') зацепит абсолютно все <img>, которые вложены в .wrapper, вне зависимости от их дальнейших вложенностей. 0

@khrustal 2 / 2 / 0 Регистрация: 26.11.2013 Сообщений: 248
	21.08.2016, 13:24 [ТС]	11
	mrtoxas, оказывается не работает в firefox, а в гуглхром нормально. Не знаете, отчего это? Кроме того, картины в новом окне не показываются оригинальным размером, а надо так. 0

@khrustal 2 / 2 / 0 Регистрация: 26.11.2013 Сообщений: 248
	21.08.2016, 14:08 [ТС]	13
	Спасибо большое, сработало. Еще если можно немного доделать. Можно ли сделать так, чтобы это новое окно 1. имело размер картины (теперь оно имеет размер картины, которое указано в html коде, а там картина показывается в уменьшенном размере) 2. окно не имело строки, где указан URL 3. окно закрывалось при клике вне пределах окна 0

Onclick для всех картин

Решение