Покритикуйте работу дилетанта

orehov · 12.04.2012, 19:40

Вот переделал сайт на дивах. Прошу оценить.

maxefect · 12.04.2012, 21:43

Ну то что вы на дивы сделали - это хорошо!
Меню на ul перевели тоже хорошо!

Осталось:
1) Доработайте композицию (цвета, тени, фон)
2) Подумайте над организацией (может вы еще лучше придумаете) - возьмите ручку и на листке рисуйте расположение блоков
3) Код! Чистить, чистить и еще раз чистить!

вот вам профессиональная документация по работе с сайтами - http://html-agency.ru/docs

PS: UDAV скоро укажет вам на детали в вашей работе! Там много неточностей, а я сказал то что нужно сделать в ближайшее будущее...

maxefect · 12.04.2012, 21:44

Замечания: Это что такое? Страница в странице?

</div>


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head>

<meta http-equiv="Content-Type" content="text/html; charset=windows-1251" />

<title>Untitled Document</title>

<link href="inkas_style.css" rel="stylesheet" type="text/css" />

</head>

<body>


=================================


</div>
</body>

</html>

</body>
</html>

Мда! Надо дорабатывать! Очень не профессионально ... вы же не хотите быть - ЛАМЕРОМ?

orehov · 12.04.2012, 21:58

Цитата:

Сообщение от maxefect

вот вам профессиональная документация по работе с сайтами - http://html-agency.ru/docs

Спасибо. Ценная для меня ссылка.

Добавлено через 4 минуты

Цитата:

Сообщение от maxefect

Замечания: Это что такое? Страница в странице?

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head>

<meta http-equiv="Content-Type" content="text/html; charset=windows-1251" />

<title>Untitled Document</title>

<link href="inkas_style.css" rel="stylesheet" type="text/css" />

Мда! Надо дорабатывать! Очень не профессионально ... вы же не хотите быть - ЛАМЕРОМ?

Да. Что-то прозевал. Поправлю

MaxSheff · 21.04.2012, 14:14

1. Фон
2. Сетка
3. Типографика

orehov · 23.04.2012, 23:19

Цитата:

Сообщение от MaxSheff

1. Фон
2. Сетка
3. Типографика

Поменять надо?

MaxSheff · 24.04.2012, 00:45

Цитата:

Сообщение от orehov

Цитата:
Сообщение от MaxSheff
1. Фон
2. Сетка
3. Типографика
Поменять надо?

1. фон поменять, по мне так несерьезно выглядит
2. Сетку надо применить, сейчас она гуляет у вас.
3. Текст немаловажная часть дизайна, также надо поработать.

by-gari · 27.04.2012, 13:55

О, знакомый логотипчик

orehov · 27.04.2012, 17:06

Ага. Благодаря Вашей помощи. Решил остановиться на этом варианте. Делал еще вот как на картинке, под другим углом, но решил что как Вы показали, будет лучше.

Похожие темы
Тема	Автор	Раздел	Ответов	Последнее сообщение
Покритикуйте набросок	Simbioziz	Скамейка запасных	17	13.02.2011 15:33
Покритикуйте	Denis	Скамейка запасных	36	17.02.2010 18:09
Покритикуйте	Scrum	Зал дебютантов	4	19.01.2007 14:32
Покритикуйте	Богдан	Зал дебютантов	24	31.10.2004 22:55
покритикуйте...	biank_a	Зал дебютантов	8	25.08.2004 12:59

12.04.2012, 19:40	#11
orehov	Вот переделал сайт на дивах. Прошу оценить.

12.04.2012, 21:43	#12
maxefect	Ну то что вы на дивы сделали - это хорошо! Меню на ul перевели тоже хорошо! Осталось: 1) Доработайте композицию (цвета, тени, фон) 2) Подумайте над организацией (может вы еще лучше придумаете) - возьмите ручку и на листке рисуйте расположение блоков 3) Код! Чистить, чистить и еще раз чистить! вот вам профессиональная документация по работе с сайтами - http://html-agency.ru/docs PS: UDAV скоро укажет вам на детали в вашей работе! Там много неточностей, а я сказал то что нужно сделать в ближайшее будущее...

12.04.2012, 21:44	#13
maxefect	Замечания: Это что такое? Страница в странице? </div> <!--начало подвала--> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <head> <meta http-equiv="Content-Type" content="text/html; charset=windows-1251" /> <title>Untitled Document</title> <link href="inkas_style.css" rel="stylesheet" type="text/css" /> </head> <body> <!-- ПОДВАЛ --> ================================= <!-- окончание подложки страницы--> </div> </body> </html> </body> </html> Мда! Надо дорабатывать! Очень не профессионально ... вы же не хотите быть - ЛАМЕРОМ? Последний раз редактировалось maxefect; 12.04.2012 в 21:50..

21.04.2012, 14:14	#15
MaxSheff	1. Фон 2. Сетка 3. Типографика

27.04.2012, 13:55	#18
by-gari	О, знакомый логотипчик