Оцените пожалуйста, первая верстка с PSD макета

[timfcsm]

страница

макет

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

чуть не забыл) вопрос 1: в панели ссылок я установил line-height для ссылок равное высоте блока, чтобы центрировать текст ссылки по вертикали, но это почему-то не срабатывает..подскажите, что тут не так

2. чую что вопрос глупый, но..маркеры перед каждой строкой в разделе портфолио, без списка как-нибудь можно сделать?

Edited January 5, 2013 by timfcsm

[Bobikys]

не все ссылки оформлены как ссылки ...

картинка где в башке лампочка наверное должна быть не фоновой а сделана как обтекаемый картинка и запилена вместе с текстом в див

2. чую что вопрос глупый, но..маркеры перед каждой строкой в разделе портфолио, без списка как-нибудь можно сделать?

вот именно список портфолие это список содержащий ссылки и маркеры там а не рамка

наверное надо списком делать

Edited January 5, 2013 by Bobikys

[RodgerFox]

Для меню исп.: line-height

в портфолио, что за бред? почему не списки?

[timfcsm]

в портфолио исправил на список ссылок..я просто сначала подумал, что раз там текст сплошняком идет значит там обычный текст)

с меню так и не понял..я и к всему блоку line-height применял, и к ссылкам..никакой реакции

[RodgerFox]

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

[timfcsm]

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

.menu {
height: 31px;
line-height: 31px;
background: url(../img/nav.png) no-repeat;
}

.menu a {
display: block;
height: 31px;
line-height: 31px;
text-transform: uppercase;
text-decoration: none;
text-align: center;
color: white;
font: bold italic 14px Tahoma, Geneva, sans-serif;
text-shadow: 1px 2px 3px #333;
}

вот так и так делаю, не работает

[RodgerFox]

для ul - overflow: hidden;

для li - line-height: 32px;

dу ссылки убрать блочность и фон выделенного меню накладывать для ли а не самой ссылки

[eis]

Почему Вы для блоков "news", "project" используете "id" а для "services" - "class"?

[timfcsm]

для ul - overflow: hidden;

для li - line-height: 32px;

dу ссылки убрать блочность и фон выделенного меню накладывать для ли а не самой ссылки

исправил..спасибо огромное)

Почему Вы для блоков "news", "project" используете "id" а для "services" - "class"?

да это я что-то клина словил) теперь у всех класс

[pincher2012]

маркер у списка не посередине. Попробуйте добавить

#container .portfolio li {

background: url(../img/marker.png) left 10px no-repeat;

}