Javascript-форум (https://javascript.ru/forum/)
-   Events/DOM/Window (https://javascript.ru/forum/events/)
-   -   Необходимо отцентрировать изображения в блоках по вертикали и горизонтали (https://javascript.ru/forum/events/65780-neobkhodimo-otcentrirovat-izobrazheniya-v-blokakh-po-vertikali-i-gorizontali.html)

kulbabka 07.11.2016 12:24
Необходимо отцентрировать изображения в блоках по вертикали и горизонтали
 
Есть такой html код:
<!DOCTYPE html>
<html>
<head>
	<meta charset="utf-8">
	<title>Example</title>
	<link media="all" rel="stylesheet" href="../common.css">
</head>
<body class="task2">

	<ul id="list-holder">
		<li><img src="images/image01.jpg" alt="image" width="132" height="92"/></li>
		<li><img src="images/image05.jpg" alt="image" width="83" height="60"/></li>
		<li><img src="images/image09.jpg" alt="image" width="90" height="83"/></li>
	</ul>

</body>
</html>


Алгоритм такой: выбрать все <li> внутри <ul id="list-holder">. В каждой найти <img>, который нужно измерить и спозиционировать по центру используя свойства marginTop и marginLeft.

Мое решение:
window.onload = function(){
	elements = document.getElementById('list-holder').childNodes;
	for(var j = 0; j < elements.length; j++){
		img = elements.getElementsByTagName('img')[j];	
	}
	for(var i = 0; i < elements.length; i++){
		img[i].style.marginLeft = ((elements[i].width - img[i].width)/2) + 'px';	
		img[i].style.marginTop = ((elements[i].height - img[i].height)/2) + 'px';
	}
};


Не срабатывает. Похоже, я неправильно ищу дочерние елементы или неправильно перебираю массив. Помогите :)

ksa 07.11.2016 13:19
Цитата:
Сообщение от kulbabka
выбрать все <li> внутри <ul id="list-holder">. В каждой найти <img>, который нужно измерить и спозиционировать по центру используя свойства marginTop и marginLeft
Центрирование можно реализовать правильной версткой. Т.о. никакой скрип не понадобится... :no:

kulbabka 07.11.2016 13:30
Тут идея с помощью js попробовать задать стили. И задание на правильное нахождение элементов

ksa 07.11.2016 14:45
Цитата:
Сообщение от kulbabka
задание на правильное нахождение элементов
Вона че...
Цитата:
Сообщение от kulbabka
Похоже, я неправильно ищу дочерние елементы
Так и есть. :yes:
Твой пример должен выглядеть как-то так...

<!DOCTYPE html>
<html>
<head>
<meta http-equiv='Content-Type' content='text/html; charset=windows-1251' />
<!--
<script src='http://code.jquery.com/jquery-latest.js'></script>
<script src="https://code.angularjs.org/1.3.9/angular.min.js"></script>
<script src="https://code.angularjs.org/1.3.9/angular-route.js"></script>
-->
<style type='text/css'>
li {
	height: 200px;
	border: 1px solid;
}
</style>
<script type='text/javascript'>
window.onload = function(){
	var img=[];
	var elements = document.getElementById('list-holder').getElementsByTagName('li');
	for(var j = 0; j < elements.length; j++){
		img[j] = elements[j].getElementsByTagName('img')[0];	
	}
	for(j = 0; j < elements.length; j++){
		img[j].style.marginLeft = ((elements[j].clientWidth - img[j].clientWidth)/2) + 'px';	
		img[j].style.marginTop = ((elements[j].clientHeight - img[j].clientHeight)/2) + 'px';
	}
};

</script>
</head>
<body>
	<ul id="list-holder">
		<li><img src="images/image01.jpg" alt="image" width="132" height="92"/></li>
		<li><img src="images/image05.jpg" alt="image" width="83" height="60"/></li>
		<li><img src="images/image09.jpg" alt="image" width="90" height="83"/></li>
	</ul>
</body>
</html>

Но это ооочень избыточный код. Ооочень много лишнего. :(

ksa 07.11.2016 14:49
Хватит и такого варианта. ;)

window.onload = function(){
	var obj = document.querySelectorAll('#list-holder img');
	for(var j = 0; j < obj.length; j++){
		var par=obj[j].parentNode;
		obj[j].style.marginLeft = ((par.clientWidth - obj[j].clientWidth)/2) + 'px';	
		obj[j].style.marginTop = ((par.clientHeight - obj[j].clientHeight)/2) + 'px';
	};
};

kulbabka 07.11.2016 15:13
Сработало так
window.onload = function(){
	var elements = document.getElementById('list-holder').children;
	for(var j = 0; j < elements.length; j++){
		var item = elements[j];
		var img = item.children[0];
		img.style.marginLeft = ((item.offsetWidth - img.width)/2) + 'px';	
		img.style.marginTop = ((item.offsetHeight - img.height)/2) + 'px';
	}
};

ksa 07.11.2016 15:28
Цитата:
Сообщение от kulbabka
Сработало так
Оно и по-моему работает... :D 
<!DOCTYPE html>
<html>
<head>
<meta http-equiv='Content-Type' content='text/html; charset=windows-1251' />
<!--
<script src='http://code.jquery.com/jquery-latest.js'></script>
<script src="https://code.angularjs.org/1.3.9/angular.min.js"></script>
<script src="https://code.angularjs.org/1.3.9/angular-route.js"></script>
-->
<style type='text/css'>
li {
	height: 200px;
	border: 1px solid;
}
</style>
<script type='text/javascript'>
window.onload = function(){
	var obj = document.querySelectorAll('#list-holder img');
	for(var j = 0; j < obj.length; j++){
		var par=obj[j].parentNode;
		obj[j].style.marginLeft = ((par.clientWidth - obj[j].clientWidth)/2) + 'px';	
		obj[j].style.marginTop = ((par.clientHeight - obj[j].clientHeight)/2) + 'px';
	};
};

</script>
</head>
<body>
	<ul id="list-holder">
		<li><img src="images/image01.jpg" alt="image" width="132" height="92"/></li>
		<li><img src="images/image05.jpg" alt="image" width="83" height="60"/></li>
		<li><img src="images/image09.jpg" alt="image" width="90" height="83"/></li>
	</ul>
</body>
</html>

Просто вариантов реализации можно сделать достаточно много...

orionpro 15.11.2016 17:22
А display: flex, разве не вариант?

ksa 16.11.2016 08:12
Цитата:
Сообщение от orionpro
А display: flex, разве не вариант?
Дык он ответил уже
Цитата:
Сообщение от kulbabka
Тут идея с помощью js попробовать задать стили. И задание на правильное нахождение элементов


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