Прошу прокомментировать верстку

[Jardineiro]

Приветствую всех! Буду очень рада замечаниям и критике верстки вот этой странички

http://www.tri-w.3dn.ru/2/index.html

Старалась добавить в нее семантику, правильно ли использованы теги html5?

[red4pony]

1) Названия классов транслитом - плохо.

2) Кнопка в навигации не отцентрованна по вертикали. Или так было в макете?

3) <div id = "footer"> --> <footer>

4) Можно использовать псевдоклассы, а не задавать лишние классы. (для IE можно подключить Selektivizir)

[klierik]

4) Можно использовать псевдоклассы, а не задавать лишние классы. (для IE можно подключить Selektivizir)

Возьму на заметку. Спасибо.

[Jardineiro]

1) Названия классов транслитом - плохо.

2) Кнопка в навигации не отцентрованна по вертикали. Или так было в макете?

3)<div id = "footer"> --> <footer>

Спасибо, поправила.

4) Можно использовать псевдоклассы, а не задавать лишние классы. (для IE можно подключить Selektivizir)

Буду разбираться, спасибо)

Edited May 24, 2013 by Jardineiro

[klierik]

1. Мне кажется тут:

a.buttn:hover {
background-position: 0 -28px;
text-decoration: none;
}

Правильнее использовать -27px, ибо высота объекта 27px и при :hover низ кнопки обрезан.

2. Почитай для чего были придуманы ID и Class. Рационально - использовать Class. Завязки стилей на ID бывает очень-очень редким исключением.

3.

Пресс-центр

Тег <P> создан что бы описывать тип данных Абзац. Я сомневаюсь что абзац может состоять из одного слова.

4.

<a href="#">New<br />Logo</a>

такие теги как <br /> в разметке появляются для форматирования контент (пользователь ввел текст через редактор ВИЗИВИГ). Верстальщик же должен избегать их. Не проблема написать: margin-bottom: высота_строки;

остальное рассматривать не вижу смысла на данном этапе. ошибок много.

[red4pony]

Пожалуйста. В книге З. Джилленуотер - Сила CSS3. Освой новейший стандарт веб-разработок (Библиотека специалиста) - объясняется как заставить практически все браузеры понимать CSS3.(есть ссылки на необходимые библиотеке) Ну и естественно описывается само применение нового языка.

1. Мне кажется тут: a.buttn:hover { background-position: 0 -28px; text-decoration: none; } Правильнее использовать -27px, ибо высота объекта 27px и при :hover низ кнопки обрезан.

ошибок много.

Магические числа

Edited May 24, 2013 by red4pony

[Jardineiro]

Правильнее использовать -27px, ибо высота объекта 27px и при :hover низ кнопки обрезан.

Тег <P> создан что бы описывать тип данных Абзац. Я сомневаюсь что абзац может состоять из одного слова.

такие теги как <br /> в разметке появляются для форматирования контент (пользователь ввел текст через редактор ВИЗИВИГ). Верстальщик же должен избегать их. Не проблема написать: margin-bottom: высота_строки;

Согласна, поправила. Спасибо!

Почитай для чего были придуманы ID и Class.

Дельный совет, почитаю.

В книге З. Джилленуотер - Сила CSS3. Освой новейший стандарт веб-разработок (Библиотека специалиста) - объясняется как заставить практически все браузеры понимать CSS3.(есть ссылки на необходимые библиотеке) Ну и естественно описывается само применение нового языка.

Да, надо набираться знаний.

[klierik]

Магические числа

и да и нет. так нарисовал кнопку дизайнер. видимо ему показалось что 30 много а 25 мало. *WALL*

[red4pony]

Я про background-position. Там изначально было можно обойтись без абсолютного позиционирования, как ТП позже и сделала (правда не лучшим образом).