Jump to content

advzr

Newbie
  • Posts

    4
  • Joined

  • Last visited

advzr's Achievements

Explorer

Explorer (1/14)

0

Reputation

  1. Спасибо за отзыв)). Меня сам дизайн интересовал больше с точки зрения верстки, а не красоты. Сперва показался простым, но, как оказалось, камни там были. Одна шапка только доставила много приятных моментов).
  2. Имена классам лучше давать такие, которые несут смысловую нагрузку и на одном языке. zagolovok и right-bar-portfolio вместе не смотрятся, а классы l и r мало что о себе говорят. Конечно для себя можно сделать как угодно, но когда захочется что-то поправить в шаблоне через год или два, придется вспоминать и ковыряться. Пустой <div class="hr"></div> ради границы - не есть хорошо. Стоило бы использовать для этого border. Сложная система у правого logo с z-index. Раз уж не захотели просто перекрашивать цвет шрифта при наведении курсора, как внизу, то можно было бы вместо img использовать фон. Тогда бы обошлись бы a { background: (рисунок с белой буквой) } и a:hover { background: (рисунок с зеленой буквой) } Не понял, зачем нужен <div class="clearfix">. Видимо он остался со времен табличной верстки. Лучше такие подчищать, так как в будущем они могут преподнести массу "приятных" моментов, когда верстка куда-нибудь уползет, и сложно будет обнаружить причину.
  3. Прошу оценить качество верстки и семантики. Есть ли какие ошибки? И самое главное, были ли моменты, которые проще или правильнее было бы сделать по-другому? Собственно сама верстка: http://www.web-advis.../first_orange2/ Шаблон взят от сюда: http://forum.htmlboo...showtopic=14398 Сделал резиновую верстку для минимальной ширины в 1024px. Код валидный. Тестировал в Firefox, Opera, Chrome, Safari 5, IE8, IE9. IE7 поддержвивается с небольшой деградацией - отсутствуют двойные кавычки в нижнем меню, и уменьшил ширину главного меню .main-nav в IE7 до 24% на элемент, так как иначе при определенных размерах ширины окна меню разваливается. Собственно большое количество вложенных тегов как раз объясняется поддержкой IE7, так как он не понимает больше одного фона для одного тега. Заранее всем отписавшимся большое спасибо.
  4. Цвет текста кнопки home не должен меняться при наведении на него курсора. Вообще правильнее, думаю, было бы сделать отдельный класс для активной кнопки, как было сделано в оригинале. Потому что когда будет выбрана, к примеру, portfolio, то придется снова делать уникальную ссылку со своей кнопкой. А так достаточно было бы только поменять класс у <li>. Как в таком случае обходиться с тем, что размер кнопок разный? Тут либо использовать http://htmlbook.ru/css/border-radius и http://htmlbook.ru/css/cat/gradient , но это поймут только новые браузеры, либо собирать кнопку из трех частей - левая и правая части с закруглениями, а также центральная часть шириной в 1px, которая дублируется по горизонтали. Левая граница между блоками webdesign и testimonials залезла на подвал. Вообще это наверное из раздела семантики и тонких материй, но я бы сделал эти границы через background, а не через img, так как смысловой нагрузки они не несут. К тому же, в таком случае граница бы обрезалась внизу и на подвал бы не наползла, даже при неправильном позиционировании. Из мелкого. Стоило заголовкам, которые сделаны через img, прописать alt="". Он у вас есть, но правильнее было бы продублировать там названия этих заголовков. Времена dial-up конечно прошли, но некоторые люди до сих пор выходят в интернет через GPRS и отключают картинки в опере. В таком случае сайт все равно останется читаемым. А в целом мне верстка понравилась.
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy