Добавила свою работу хотелось бы услышать объективный комментарий по верстке Портфолио
Оценить: + | 0 Комментировать
Раздел: Обучение    
Теги:
Просмотров: 220

Комментарии (11):

0
Иван Кирюхин [kendrive] [24 января 13: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»

В целом двигаетесь в верном направлении, но нужно пытаться верстать более сложные макеты.
0
[Sidoruk-SV] [24 января 14:04]
Указывать полный путь к стилю элемента это уж изварщение как по мне.
Если уж начали про БЭМ, то думаю стоило дать ссылку(внизу страницы есть доп. линки) и ещё раз перечитать, стили к "независимым блокам" не могут содержать .wrapper по определению
0
Григорий Кочнев [Greg] [24 января 13:44]
В общем не плохо, но есть несколько мелких замечаний:
1. Используйте классы вместо ID. Айдишники используются в основном для скриптов. Плюс оставите себе возможность использовать блок несколько раз на странице.
2. Старайтесь давать логическое название классам. Например: вместо "fl_left" можно было "logo".
3. Не пишите стили в ХТМЛ коде(style="left: 0pt;"), для этого есть CSS.
4. Указывайте размеры картинок
0
Балицкая Татьяна [inno1876] [24 января 13:49]
Вопрос о шрифтах. Trebuchet MS разве является не стандартным?
0
[Sidoruk-SV] [24 января 13:58]
"work on Windows and MacOS but not Unix+X" (Ubuntu, Gentoo)
0
Балицкая Татьяна [inno1876] [24 января 14:05]
Ок сп
0
Балицкая Татьяна [inno1876] [29 января 10:42]
Добрый день, подскажите, подключила не системный шрифт GaramondPremrPro-ItDisp он по разному отображается в разных браузерах есть ли возможность его использования ?
0
[Sidoruk-SV] [29 января 18:30]
Что вы имеете ввиду под "отображается по разному в разных браузерах"?
Какой сервис использовали для обеспечения кроссбраузерности (советую fontsquirrel.com/fontface/generator)
0
Балицкая Татьяна [inno1876] [05 февраля 18:49]
Обновила портфолио
0
Балицкая Татьяна [inno1876] [вчера в 1:00]
Добрый вечер.В макете SmallSpeakers есть проблема, при наведении мышки на главное меню смещается весь блок, в браузерах опера и мазила. В сафари и в гугле свойство а hover не работает. подскажите как решить эту проблему?. Спасибо.
0
[Sidoruk-SV] [вчера в 1:32]
Уберите в
.right_part li a:hover {
padding: 10px;
}


И разберитесь почему у вас в хроме не работает :hover (если в dev tools жестко указать состояние hover http://d.pr/2pHN, то тоже прыгает, так как стиль так указан).

Написать комментарий

Комментарии к статье может оставить только зарегистрированный пользователь.


Вы можете также оставлять комментарии, авторизовавшись в одной из социальных сетей: