Нужна оценка, критика, советы

YanaSt · August 30, 2009

В общем, первый сайт, сверстанный мною.

Был макет в psd.

Просьба указать на ошибки, дать общую оценку ВЕРСТКИ.

Там еще два скрипта подключила, просто подсмотрела на других сайтах как это делается, внутри ничего в них не меняла, потому что пока только изучаю HTML и CSS, в javascript вообще ничего не смыслю.

И еще, не проходит валидатор, но не пойму в чем дело, ругается на </body>, </html>.

Мое мнение (если не брать во внимание тормознутость скриптов): вся конструкция тяжеловата, наверное, код можно было упростить. Ну первый блин, в общем.

Адрес сайта: stepalex.ru

To psywalker: полоску "заборола" как смогла :-)

Lionel · August 30, 2009

В FF у меня сильно тормозит и дергается прокрутка, в опере этого нет. В навигации в опере отсутствуют пунктирные строки, как в FF. И фон... опять же, в FF он текстурированный, а опере сплошной...

Ну а в IE у меня вот такое -

YanaSt · August 30, 2009

а у меня все нормально отображается

странно ((((

Glebr · August 30, 2009

В общем, первый сайт, сверстанный мною.

Для первого сайта не плохо, но слишком уж длинная страница...

Great Rash · August 30, 2009

обновлено: 27.08.09 статус: Свободен

Это ужас!

<img src="images/polosa.png" alt="" />

С какой целью понадобилось заключать картинку в параграф?

<a href="http://stepalex.ru/"><img src="images/logo.png" alt="" /></a>

То же самое со ссылкой

<table class="table_portfolio" border="0" width="" cellpadding="0" cellspacing="0">

Про CSS вы не помните совершенно.

<!—САЙТЫ-->

Comments on Russian - not very good

Таблицы... а начиналось все тах хорошо.

Кодировка windows-1251 - не кошерно.

Сам по себе код "неопрятен" и "грязен"...

Lionel · August 30, 2009

а у меня все нормально отображается
странно ((((

Я проверял в IE6, новее не ставлю, потому что вообще ослом не пользуюсь.

YanaSt · August 31, 2009

обновлено: 27.08.09 статус: Свободен
Это ужас!

Вы напишите, что именно Вас так ужаснуло. Как-то неконструктивно получается. Критика есть, а что именно критикуется и как исправить - нет. Ценности никакой от таких реплик.

Так понимаю, куча пробелов? Или что-то еще? Чем заменить? Пустым дивом?

Цитата

<img src="images/polosa.png" alt="" />

С какой целью понадобилось заключать картинку в параграф?

Валидатор ругается, если картинка не в блочном элементе. Нужно в div заключать?

Цитата
<table class="table_portfolio" border="0" width="" cellpadding="0" cellspacing="0">
Про CSS вы не помните совершенно.

Да, здесь забыла.

Цитата
<!—САЙТЫ-->
Comments on Russian - not very good

Не знала, буду иметь ввиду

Таблицы... а начиналось все тах хорошо.

Предложите Ваш способ упорядочить превьюшки разной высоты с выравниванием по верхнему краю. Разве в данном случае таблица - зло?!?!

Кодировка windows-1251 - не кошерно.

Так напишите как кошерно ))) Я смотрела, как сделано на этом сайте, вроде бы так же.

Сам по себе код "неопрятен" и "грязен"...

Ну это вообще ничего мне не дает, что исправлять-то??

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

Я проверял в IE6, новее не ставлю, потому что вообще ослом не пользуюсь.

Да, я в 7 смотрела.

Для первого сайта не плохо, но слишком уж длинная страница...

Спасибо! Страница длинная, потому что пока не знаю, как сделать листаж страницы Как узнаю, разобью все разделы по страницам, а на каждой странице будет не более 3 рядов с превьюшками.

BassEast · August 31, 2009

всё хорошо, леди ; )

Вы правильно мыслите в плане вёрстки, остаётся набраться опыта.

Игорь Ермаков · August 31, 2009

Кто там говорил про таблицы?

Таблицы в данном случае абсолютно приемлемы, так как данные именно табличные.

А так всё достаточно аккуратно и культурно.

YanaSt · August 31, 2009

ой, бальзам на душу

Great Rash · August 31, 2009

YanaSt

Вы напишите, что именно Вас так ужаснуло. Как-то неконструктивно получается. Критика есть, а что именно критикуется и как исправить - нет. Ценности никакой от таких реплик.
Так понимаю, куча пробелов? Или что-то еще? Чем заменить? Пустым дивом?

Да, куча пробелов. Для этих целей существует margin и padding.

Валидатор ругается, если картинка не в блочном элементе. Нужно в div заключать?

Полосу огромную можно положить на бекграунд в див. Параграф нужен для хранения текста, а не элементов дизайна. Слово "Портфорио", надо тоже на бекграунд класть. Запомните, элементы дизайна ни в коем случае нельзя делать через <img>, их всегда надо стараться делать бекграундом через стили.

Да, здесь забыла.

Ай-яй-яй, это надо знать как "Отче наш" - текст на HTML, оформление на CSS. В HTML оформления не должно быть вообще.

Не знала, буду иметь ввиду

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

Так напишите как кошерно ))) Я смотрела, как сделано на этом сайте, вроде бы так же.

Кошерно ставить кодировку UTF-8 всегда и везде. Т.к. она работает всегда и везде.

UPD:

Предложите Ваш способ упорядочить превьюшки разной высоты с выравниванием по верхнему краю. Разве в данном случае таблица - зло?!?!

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
	<title>template</title>
	<meta http-equiv="content-type" content="text/html;charset=utf-8" />
	<meta http-equiv="Content-Style-Type" content="text/css" />

	<style type="text/css">
		div {
			display: inline-block;
			width: 100px;
			border: 1px solid;
		}

		.h1 {
			height: 100px;
		}

		.h2 {
			height: 200px;
		}

		.h3 {
			height: 300px;
		}

		.h4 {
			height: 400px;
		}
	</style>
</head>
<body>

	<div class="h1">text</div>
	<div class="h2">text</div>
	<div class="h3">text</div>
	<div class="h4">text</div>
	<div class="h1">text</div>
	<div class="h2">text</div>
	<div class="h3">text</div>
	<div class="h4">text</div>
	<div class="h1">text</div>
	<div class="h2">text</div>
	<div class="h3">text</div>
	<div class="h4">text</div>
	<div class="h1">text</div>
	<div class="h2">text</div>
	<div class="h3">text</div>
	<div class="h4">text</div>

</body>
</html>

Если вы вдруг захотите чтобы у вас на одной строке помещалось не 3 превьюшки а пять, или наоборот 2, то вам придется переписывать код таблицы. В моем же случае мне достаточно переписать стили. Плюс в моем случае я не привязан к ширине таблицы и при изменении ширины окна контейнеры сами выстраиваются по нужному количеству. Соответственно на широких экранах высота страницы будет меньше (а она у вас не маленькая, особенно без постраничной навигации).

UPD2: По поводу неопрятного кода я бессилен, что либо объяснить - тут чувствовать надо. Может и с опытом приходит... хз.

Edited August 31, 2009 by Great Rash

cawa · September 12, 2009

мдя! :rolleyes:

присутствуют следы копирования!

в особености то что почти все параметры alt пусты!

<!—конец НАРУЖКА-->        

<!—ВВЕРХ-->

ну хоть каментарии не пусты

Victor84 · September 18, 2009

мне нравится

Clayton · September 25, 2009

Дизайн мне понравился, а верстка нед. Даже такие превью отлично верстаются без таблиц)

Keyv · October 12, 2009

Страница в целом весит 394 Кб. По-моему, многовато. Да и длинная чересчур: прокручивается в 7 раз...

Павел Билькис · October 12, 2009

YanaSt, вы мужчина?

Edited October 12, 2009 by Павел Билькис

YanaSt · October 12, 2009

а с чего вдруг такой вопрос??

Justnewone · October 30, 2009

а с чего вдруг такой вопрос??

И еще, добавлю пять копеек. простые стили не стоит вешать на id, помянете мое слово, если в верстке с id стилями надо будет поменять оформление.

SelenIT · October 30, 2009

</table>
<td valign="top" width="310px">

1) где закрывающая </td>? Раз уж стоит доктайп XHTML, так надо соблюдать веллформность...

2) width="310px" — какой это язык?

YanaSt · October 30, 2009

И еще, добавлю пять копеек. простые стили не стоит вешать на id, помянете мое слово, если в верстке с id стилями надо будет поменять оформление.

что есть "простые стили"?

1) где закрывающая </td>? Раз уж стоит доктайп XHTML, так надо соблюдать веллформность...
2) width="310px" — какой это язык?

про закрывающий тег - спасибо, учту

а по второму пункту что не так?

SelenIT · October 30, 2009

CSS-ная размерность в HTML-ном атрибуте. В HTML нет единицы "px", там пиксели по умолчанию. В CSS единица измерения нужна, но синтаксис другой. Лучше, конечно, задавать размеры именно стилями.

YanaSt · October 30, 2009

понятно... спасибо

Searcher · November 2, 2009

про закрывающий тег - спасибо, учту
а по второму пункту что не так?

все оформление в CSS, нет таких атрибутов уже давно!

roman3211 · May 12, 2010

прикольный сайт, графики многовато, но в целом дизайн мне понравился

Searcher · May 13, 2010

прикольный сайт, графики многовато, но в целом дизайн мне понравился

Я рад за вас! Но зачем поднимать старые темы?

Sign In

Нужна оценка, критика, советы

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Обсуждения

Important Information