Моя первая верстка. Больше критики)

[ikilldog]

Всем привет!

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

http://ikilldog.narod2.ru/frfrf/index.html

http://ikilldog.narod2.ru/frfrf/simple.jpg - а вот сама изначальная картинка

Edited October 7, 2012 by ikilldog

[cyklop77]

курсор при наведении на пункт меню почему как для ввода текста?

[ikilldog]

в меню просто не стал делать ссылки, так что видимо из-за этого

[amelice]

Всем привет!

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

http://ikilldog.narod2.ru/frfrf/index.html

http://ikilldog.narod2.ru/frfrf/simple.jpg - а вот сама изначальная картинка

1. Если указали рамку для контента, надо было её отобразить? Или для чего она нужна?

margin: auto - вполне достаточно, так как верхний и нижний margin сбрасывается в 0 при auto.

Высоту для контента явно указывать тоже не стоит, для особых случаев есть min-height.

.content {
    margin: 0 auto;
	height: 666px;
	border: 4px;
	border-color: #ffffff;
}

2. Нужно подумать для чего нужна навигация или посмотрите как она оформлена на других сайтах.

<div class="menu">
		<span class="um">UM</span>
                  ...
	</div>

3. Правильно организуйте селекторы и напишите правила для них, чтобы не повторять.

.album  {
	....
}
.album:hover {
	..
}
.tech  {
	...
.tech:hover {
	...
}

4. Это свойство только для элементов списка.

.album  {
	list-style-type: none;
}

5. В вашем случае абсолютное позиционирование в контенте для элементов вовсе не нужно.

6. Картинку желательно сделать бэкграундом, т.к. она не несет никакой смысловой нагрузки

7. Если не собираетесь менять цвет фона, не нужно писать background-color: transparent, т.к. это значение по умолчанию для данного свойства.

footer {
	background-color: transparent;

8. Не нужно для каждого элемента писать свойство font. Это свойство наследуется, поэтому укажите для body один раз.

Edited October 11, 2012 by amelice

[kompolom]

Шрифт покрупнее чуток. А то сливается.

[ikilldog]

Всем большое спасибо за замечания! В ближайшее свободное время сяду переделывать как следует

Edited October 16, 2012 by ikilldog