[umkasanki]

Вот сайтик небольшого магазина сладостей
http://chocoroom.by/
оцените пожалуйста
п.с. просьба сразу не плеваться на фон сайта, хотелось создать ошущение уюта, ассоциации со старой упаковочной бумагой...

[Line]

> хотелось создать ошущение уюта
Вам удалось его просрать. цветовая гамма ужасная. воспользуйтесь kuler.adobe.com. рассмотрите иные шрифтовые решения.

Вашим сайтом относительно удобно пользоваться, но дизайн просто никакой. впрочем, для начала 00-х вполне сойдёт.

посмотрите на другие сайты схожей тематики.

порой стоит воспользоваться готовыми шаблонами.

[Silver]

Дизайн не современный - это да, но в целом вроде сайт свою функцию выполняет.

[maxefect]

Хотели создать ощущение уюта! А получился фильм ужасов (фон)
Пока фон не загрузился, я думал норм будет ... а нет оказывается

1. Проверил на слабом соединении.. долго грузится картинка в шапке ...

2. Меню бы нормально бы сделать (шрифт и тд), чтобы норм. смотрелось

3. Над шрифтами поработайте (где то курсив не к месту - " Каталог ")

Добавлено через 43 минуты

Теперь буду разносить вас по коду

1. Выполните нормальную табуляцию

2. CSS / А вот объясните такой казус!

В основном стиле main.css у вас стоит:

@import "reset.css" screen;
@import "colorsAndBg.css" screen;
@import "typography.css" screen;
@import "html5fix.css" screen;
и тд

То есть вы уже их подключили в главную таблицу стилей! Но при этом у вас в HTML документе стоит:

<!--[if lt IE 9]> 
        <link type="text/css" rel="stylesheet" media="all" href="http://chocoroom.by/chocoroom/css/reset.css"/>
        <link type="text/css" rel="stylesheet" media="all" href="http://chocoroom.by/chocoroom/css/html5fix.css"/>
        <link type="text/css" rel="stylesheet" media="all" href="http://chocoroom.by/chocoroom/css/colorsAndBg.css"/>    
        <link type="text/css" rel="stylesheet" media="all" href="http://chocoroom.by/chocoroom/css/jquery.tooltip.css"/>
        <link type="text/css" rel="stylesheet" media="all" href="http://chocoroom.by/chocoroom/css/ie.css"/>

То есть вы их снова переподключаете! Вы ведь когда заходите через IE, у вас ведь стиль main.css загружается, в котором уже загружены все эти стили reset.css и тд .... Это плохо и это создает некий глюк! Уберите строчки кода со стилями <!--[if lt IE 9]> <link type="text ...

Конечно, оставьте закомментированным <!--[if lt IE 9]> <script src="http://html5shiv.googlecode.com/svn/trunk/html5.js"></script> - это!

В main.css

зачем @import "" screen; ?

3. Много стилей!

Ну, @import конечно работает быстро! Но в IE оно тормозит ... вообще все стили можно объединить в один (меньше подключений к серверу)

4. JavaScript

У вас многое тупо вынесено в <script> БЛА БЛА БЛА </script>
Выносите в отдельный файл! И подключайте ....

5. Упрощение для тегов

... у вас ведь Doctype -> HTML5

<link type="stylesheet"
type="stylesheet" - можно упускать
<script type="javascript"
type="javascript" - можно упускать

6. Ненужные комментарии

Зачем вам в странице после каждого тега, который закрывается комментарий о том что он закрывается?
<!--end of div class="price"--> - в тем более особого смысла это не дает!
Если бы у вас все было бы грамматно табулировано, было бы понятно

7. Скрипт Fancybox

У вас картинки по нажатию загружаются! Приходится ждать ... а пользователю было бы интересно на них сразу посмотреть!

Поэтому легче их подгружать заранее! Маленький код предзагрузки картинки (к примеру)

<sript src="preload.js"></script>
в нем:
pic = new Image();
pic.src="test.jpg";

И когда зайдут на сайт, она уже будет подгружена! И по нажатию на картинку (где ссылка <a href="test.jpg" class="fancybox">) ее не надо грузить...