Просьба оценить верстку

[roman_kr_ru]

Здравствуйте!

Прошу оценить сайт: http://organic.esy.es 

С удовольствием выслушаю Ваши замечания.

 

Файлы и макет .psd: http://rghost.ru/50491074

[Ludwig Voltman]

1) не прописана ширина у враппера (body). Если сайт должен тянуться, то необходимо прописать параметры min-width, max-width. Сейчас при изменении ширины окна браузера дизайн разваливается.

2) логотип я бы сделал флоатом, а не позиционировал абсолютно. Сейчас при изменении ширины экрана панель навигации налезает на логотип

3) не совсем понял организацию стилей в области слайдера и навигации. Одни и те же стили написаны и в html (инлайн), и в css. Зачем? Можно переместить инлайновые стили в css файл?

4) цитату в слайдере я бы сделал через blockquote, а не через div. Почему кавычки сделаны через рисунок, да ещё и img?

5) почему background в блоке footer сделан через фоновую картинку, если там однотонная заливка (background-color)? Почему формат для однотонной фоновой картинки jpg, а не gif/png8?

Edited November 28, 2013 by Ludwig Voltman

[roman_kr_ru]

1) не прописана ширина у враппера (body). Если сайт должен тянуться, то необходимо прописать параметры min-width, max-width. Сейчас при изменении ширины окна браузера дизайн разваливается.

Изменил.

логотип я бы сделал флоатом

Попробовал флоат. Все равно наезжает на меню.

Одни и те же стили написаны и в html (инлайн), и в css

В html  нет атрибута style, я еще раз проверил даже. Или что Вы имеете ввиду? 

через blockquote

Да, согласен, так правильнее.

Почему кавычки сделаны через рисунок

Основной шрифт, который я подключил через гугл-фонтс имеет совсем другие кавычки, поэтому сделал через рисунок. Хотя, сейчас понял, что нужно было использовать :before и :after.

 

однотонная заливка

Там "шумы" какие-то.

не gif/png8

Моя невнимательность.

[Ludwig Voltman]

В html нет атрибута style, я еще раз проверил даже. Или что Вы имеете ввиду?

1. <div class="bx-wrapper" style="max-width: 100%; margin: 0px auto;">

2. <div class="bx-viewport" style="width: 100%; overflow: hidden; position: relative; height: 300px;">

3. <ul class="bx-slider" style="width: 415%; position: relative; transition-duration: 0s; transform: translate3d(-325px, 0px, 0px);">

4. <li class="bx-clone" style="float: left; list-style: none outside none; position: relative; width: 325px;">

Эти inline-стили можно вынести в css файл, чтобы они не занимали место в html-коде.

Основной шрифт, который я подключил через гугл-фонтс имеет совсем другие кавычки, поэтому сделал через рисунок. Хотя, сейчас понял, что нужно было использовать :before и :after.

 

Посмотрел макет - кавычки сделаны тем же шрифтом (lato). Это обычные кавычки (которые shift+2), размером 48 пунктов (где-то 48 пикселов).

[roman_kr_ru]

 

1. <div class="bx-wrapper" style="max-width: 100%; margin: 0px auto;">

2. <div class="bx-viewport" style="width: 100%; overflow: hidden; position: relative; height: 300px;">

3. <ul class="bx-slider" style="width: 415%; position: relative; transition-duration: 0s; transform: translate3d(-325px, 0px, 0px);">

4. <li class="bx-clone" style="float: left; list-style: none outside none; position: relative; width: 325px;">

Эти inline-стили можно вынести в css файл, чтобы они не занимали место в html-коде.

Для слайдера я использовал плагин bx-slider: видимо, эти дивы и стили создаются самим плагином через DOM. Их можно как-то вынести?

[Николя223]

тут оценивать нечего, на самом деле...  слишком мало всего.  Если бы вы закреативили - дизайн, тогда минималистичность было бы плюсом 

[Ludwig Voltman]

Для слайдера я использовал плагин bx-slider: видимо, эти дивы и стили создаются самим плагином через DOM. Их можно как-то вынести?

 

Не знаю, никогда не работал со слайдерами. Лучше вынести - код уродский получается. Но если нельзя - придется оставить.

[Z1gnet]

Не знаю, никогда не работал со слайдерами. Лучше вынести - код уродский получается. Но если нельзя - придется оставить.

я код не открывал (нету возможности), но больше чем уверен что  эти стили довешиваются динамически джаваскриптом. это обычное дело, с джеесом так всегда. и ничего тут он не поделает и ничего страшного в этом нету