<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
— устарел, используй
<!doctype html>
Не используй для стилей id, только классы. id нужно оставить для скриптов
Для отделения хедера, футера нужно использовать классы footer и header или производные от них. А не wrapper_col
Шрифты не стандартные, если хотите их использовать, то их необходимо специальным образом подключать.
По реже использовать наследование, делать все блоки стиле-независимыми и желательно писать не ul#top_nav li, а полный путь «.wrapper_col .top_menu .fl_right ul#top_nav li»
В целом двигаетесь в верном направлении, но нужно пытаться верстать более сложные макеты.
Указывать полный путь к стилю элемента это уж изварщение как по мне.
Если уж начали про БЭМ, то думаю стоило дать ссылку(внизу страницы есть доп. линки) и ещё раз перечитать, стили к "независимым блокам" не могут содержать .wrapper по определению
В общем не плохо, но есть несколько мелких замечаний:
1. Используйте классы вместо ID. Айдишники используются в основном для скриптов. Плюс оставите себе возможность использовать блок несколько раз на странице.
2. Старайтесь давать логическое название классам. Например: вместо "fl_left" можно было "logo".
3. Не пишите стили в ХТМЛ коде(style="left: 0pt;"), для этого есть CSS.
4. Указывайте размеры картинок
Добрый день, подскажите, подключила не системный шрифт GaramondPremrPro-ItDisp он по разному отображается в разных браузерах есть ли возможность его использования ?
Что вы имеете ввиду под "отображается по разному в разных браузерах"?
Какой сервис использовали для обеспечения кроссбраузерности (советую fontsquirrel.com/fontface/generator)
Добрый вечер.В макете SmallSpeakers есть проблема, при наведении мышки на главное меню смещается весь блок, в браузерах опера и мазила. В сафари и в гугле свойство а hover не работает. подскажите как решить эту проблему?. Спасибо.
И разберитесь почему у вас в хроме не работает :hover (если в dev tools жестко указать состояние hover http://d.pr/2pHN, то тоже прыгает, так как стиль так указан).
Комментарии (11):
<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"><!doctype html>Не используй для стилей id, только классы. id нужно оставить для скриптов
Для отделения хедера, футера нужно использовать классы footer и header или производные от них. А не wrapper_col
Шрифты не стандартные, если хотите их использовать, то их необходимо специальным образом подключать.
По реже использовать наследование, делать все блоки стиле-независимыми и желательно писать не ul#top_nav li, а полный путь «.wrapper_col .top_menu .fl_right ul#top_nav li»
В целом двигаетесь в верном направлении, но нужно пытаться верстать более сложные макеты.
Если уж начали про БЭМ, то думаю стоило дать ссылку(внизу страницы есть доп. линки) и ещё раз перечитать, стили к "независимым блокам" не могут содержать .wrapper по определению
1. Используйте классы вместо ID. Айдишники используются в основном для скриптов. Плюс оставите себе возможность использовать блок несколько раз на странице.
2. Старайтесь давать логическое название классам. Например: вместо "fl_left" можно было "logo".
3. Не пишите стили в ХТМЛ коде(style="left: 0pt;"), для этого есть CSS.
4. Указывайте размеры картинок
Какой сервис использовали для обеспечения кроссбраузерности (советую fontsquirrel.com/fontface/generator)
.right_part li a:hover {
padding: 10px;
}
И разберитесь почему у вас в хроме не работает :hover (если в dev tools жестко указать состояние hover http://d.pr/2pHN, то тоже прыгает, так как стиль так указан).
Написать комментарий
Вы можете также оставлять комментарии, авторизовавшись в одной из социальных сетей: