yafilip
-
Posts
4 -
Joined
-
Last visited
Content Type
Profiles
Forums
Calendar
Store
Posts posted by yafilip
-
-
Весьма неплохо для начинающего. Кроме тех моментов что уже указал NeoXidizer, хочу добавить что тег nav тем и удобен что нет необходимости плодить ul-li - пишите просто:
<nav>
<a href="#">...</a>
<a href="#">...</a>
<a href="#">...</a>
<a href="#">...</a>
</nav>Ну и если добавить HTML5 Shiv вместе с PIE будет и в ослах все замечательно
сделал как вы сказали. Но теперь не могу понять, через какой элемент указать расстояние между ссылками ан навигаторе?
Вернее понимаю, через #top_navigator a {}, но удобнее было через li, т.к. кнопки прыгают по панеле сейчас как сумасшедшие )
-
Смотрю только техническое исполнение:
1. В сайдбаре к textarea добавьте css атрибуты { overflow: auto; resize:none; }, таким образом уберется скроллбар в IE и кнопка ресайзинга в WebKit браузерах
2. Не используйте изображения большого размера, если они "сжимаются" на странице - ваше лого в оригинале 1181 ? 788 - 35кб, нужно уменьшить размер, например пропорционально обрезав в фотошопе
3. Вы используете тег <br\> там, где нужен отступ от элемента - margin
4. Укажите для кнопки Send it css атрибут cursor:pointer;, чтобы показать, что эта кнопка совершает действие и на нее необходимо нажать
5. Оберните пояснения к полям в <label>
например:
E-mail: <input type="text" class="field" id="email" name="email" />
преобразуется в
<label for="email">E-mail: </label><input type="text" class="field" id="email" name="email" />
6. Укажите для label атрибут cursor:pointer; (хотя зависит от дизайна и "кому как нравится")
7. Не используйте глобальный селектор, это плохо сказывается на скорости рендинга (прорисовки) страницы, особенно если много контента, замените
* { margin: 0; padding: 0; }на
html,body,div,p,a,img,header,nav,ul,li,aside,h1,form,label,input,textarea,section,footer {margin:0;padding:0}дополняя элементами по мере наполнения страницы
8. Замените
<meta content="text/html; charset=utf-8" http-equiv="Content-Type" />
на
<meta charset="utf-8" />
и поместите этот код сразу после открытия <head>
8. Добавьте HTML5 Shiv скрипт для поддержки IE<9 http://code.google.com/p/html5shiv/
спасибо большое, хорошие замечания, все свои ошибки понял, исправлю!
-
Вот собственно сам сайт. Жду критику и рекомендации, пожалуйста.
Покритикуйте. Только начал работу в этой сфере
in Discussion of works
Posted
заметил баг кстати говоря. Когда жмешь по ссылке в навигаторе, как бы чуть ниже, под текстом, то она нажимается, но не выполняет никакого действия. Что это такое? )