Еще одна Раира

[harrids]

Сабж.

Макет здесь: http://forum.htmlbook.ru/index.php?showtopic=13276

Результат тут: http://harrids.com/projects/raira/

Замеченные проблемы:

1. Текст про консультации при наезде на перо становится трудночитаемым. Можно было добавить ему фона или сдвинуть куда-нить, но в обоих случаях выглядит убого, так что оставил как есть).

2. Опять же трабла с верхним текстом - если достаточно сильно увеличить размер шрифта, текст выезжает на нижний блок. Хотел было сделать шапку полностью эластичной, но это большая херь (чернильница) плохо стыкуется с окружающим миром, плюс при использовании пнг с прозрачностью это вылилось бы ~ в 150-200кбайт. Хотяяя, можно было и засунуть ее в <img> и ресайзить вместе с текстом, но как то не хотелось) Приемлемых же способов ресайзить бэкграунд я не нашел, если знаете - поделитесь.

3. Если нету шрифта Monotype Corsiva, будем наблюдать Comic Sans . При этом заголовок "Наш адрес" не влезет в одну строку, а под лупой по-прежнему будет Monotype Corsiva

Вроде ничего не забыл. Приму любую критику)

[Павел Билькис]

хорошая верстка.

[psywalker]

Ну вообще я тебе скажу, что ты выполнил работу лучше, чем твои предшественники, и это радует! Но без ошибок конечно-же тоже не обошлось

1) Шрифты во многих местах не соответствуют макетным

2) В шапке крассный блок в шапке, где логотип - должна переливаться, а у тебя сплошной цвет

3) Чернильница должна быть Пнг-24 и от неё должна отходить тень

4) Заголовки: Скидки и Наши люди надо было всё таки делать картинками, а то некрасиво, когда каждое слово находится на разных строчках, да ещё и шрифт не совпадает

5) Вот эта вещь совершенно ни к чему: class="clearfix" - постарайся исключить его из будущих работ

6) Каркас центра сделан неправильно, в данной вёрстке можно сделать всё намного прощее

7) В ИЕ7 при сужении баннеры съезжают вниз, потом при расширении экрана сайт зависает на определённой ширине и не растягивается, как надо

8) В ИЕ6 не стоит минимальная ширина и сайт резиновый

Так, ну на первый взгляд вроде всё, могу сказать, что понравилось:

1) Ты сделал реловер на баннеры, молодец! Ты первый, кто это сделал

2) Бабочку положил в правильную колонку и правильно применил к ней стили

3) Затратил немного элементов div и других

3) И вообще ты молодец, видно что старался, продолжай в том-же духе

Edited July 11, 2009 by psywalker

[harrids]

Павел Билькис, psywalker, спасибо!

psywalker'у отдельное спасибо за развернутый комент)

Ну и по пунктам:

1) Если честно, в упор не вижу где. Единственное, где отошел, это заголовкам в правой колонке сделал 48px, чтобы с текстом под лупой одинаково было.

2) Да кстати, забыл про это упомянуть) Градиент там какой-то странный - вырезать так, чтобы замостилось без стыков, проблематично. А всю полоску вырезать как-то некошерно) Поэтому остановился на таком варианте.

3) Последствия оптимизации) Имхо, 170кбайт экономии стоили того. Плюс, не понял про тень - вроде все в соответствии с макетом.

4) Ооочень долго размышлял на эту тему, но так и не смог заставить себя делать самый обычный текст картинками. Когда вижу подобные вещи, внутри меня что-то умирает

В принципе, можно подобрать более подходящий шрифт на замену, но я в типографии не силен)

5) Почему? Имхо, блок с флоатами не должен надеяться на то, что кто-то после него сделает clear.

6) А пример?)

7) Вроде пофиксил, про расширение не понял, вроде нормально было. Кстати, пока фиксил, заметил другой баг - если сделать размер текста меньше среднего, то при растягивании страницы больше max-width'a начинает двигаться футер) После обновления страницы встает на место.

8) Да, долго искал подходящего решения, но так и не нашел - ввиду того, что колонки переставляются местами отрицательными маргинами, постоянно возникают непонятные для меня траблы) В общем, надо будет поплотнее поизучать проблему)

[psywalker]

1) Большинство пунктов посмотри тут: http://psywalker.ru/Raira/main-page.html

2)

5) Почему? Имхо, блок с флоатами не должен надеяться на то, что кто-то после него сделает clear.

Никаких ИМХО, либо ставь overflow: hidden; родителю, либо верстай так, что-бы потом проблем не возникало. Приучи себя к этому.

3)

6) Каркас центра сделан неправильно, в данной вёрстке можно сделать всё намного прощее

Так сразу не объяснишь - это придёт со временем, если хочешь, постучи ко мне в аську, я тебе покажу пару приёмов)))

4)

Вроде пофиксил, про расширение не понял, вроде нормально было. Кстати, пока фиксил, заметил другой баг - если сделать размер текста меньше среднего, то при растягивании страницы больше max-width'a начинает двигаться футер) После обновления страницы встает на место.

Повесь на body - position: relative; и всё будет норм!

5)

8) Да, долго искал подходящего решения, но так и не нашел - ввиду того, что колонки переставляются местами отрицательными маргинами, постоянно возникают непонятные для меня траблы) В общем, надо будет поплотнее поизучать проблему)

Забудь про отрицательные Маргины - это старые методы, опятьже уже давно делается всё намного проще и качественнее

[Phantom]

Вёрстка в принципе не плохая,только вот дизайн всё таки должен быть резиновый,логотип не переливается,в нижней части шапки,там где ссылки,фон должен быть белым как на макете,а не с градиентом(такие вольности дизайнеры обычно не очень любят ).Вот вообще-то и все замечания)

Порадовал небольшой структурированный код и ролловер на баннерах)

Никаких ИМХО, либо ставь overflow: hidden; родителю, либо верстай так, что-бы потом проблем не возникало. Приучи себя к этому.

overflow: hidden; не всегда полезен,в некоторых случаях всё таки лучше добавить блок с clear: both;

[psywalker]

overflow: hidden; не всегда полезен,в некоторых случаях всё таки лучше добавить блок с clear: both;

Надо действовать по уму и чувствовать ситуацию, где надо вставить clear: both; а где overflow: hidden; , а чутьё приходит с опытом! В любом случае любой мусор наподобие этого ужаса class="clearfix" и div class="clear" не должен присутствовать в коде верстальщика, который понимает и любит своё дело.

[harrids]

Не согласен насчет "ужасов" class="clearfix" и иже с ними. Не увидел ни одного конкретного аргумента против, одни эмоции и голословные утверждения.

Зачем нужны селекторы классов? Чтобы улучшить reusability кода. В чем смысл задавать, скажем, десятку элементов одно и то же правило, плодя тем самым не нужный код? А если я захочу его потом скорректировать? Бегать по всем? Ctrl+H? А ведь при использовании классов все правится в одном месте.

Можно группировать селекторы, скажите вы? Да, можно, отвечу я) Тут уже на вкус и цвет.

В общем, либо конкретные drawback'и, либо didn't happen.

[psywalker]

harrids

Не согласен насчет "ужасов" class="clearfix" и иже с ними. Не увидел ни одного конкретного аргумента против, одни эмоции и голословные утверждения.

Зачем нужны селекторы классов? Чтобы улучшить reusability кода. В чем смысл задавать, скажем, десятку элементов одно и то же правило, плодя тем самым не нужный код? А если я захочу его потом скорректировать? Бегать по всем? Ctrl+H? А ведь при использовании классов все правится в одном месте.

Можно группировать селекторы, скажите вы? Да, можно, отвечу я) Тут уже на вкус и цвет.

Я своё мнение высказал, а дальше твои проблемы, как ты будешь действовать