Оцените уровень и качество работы

[FaLaRD]

Всем привет

Оцените пожалуйста насколько хорошо выполнена работа, и что можно было бы сделать чтобы она была выполнена лучше.

http://falard.org/chapter-two.html

Кроссбраузерность IE+7

Буду рад любой конструктивной критике.

Спасибо!

[cris7iano]

Edited October 18, 2013 by cris7iano

[FaLaRD]

Дружище ты ничего не перепутал? ID у меня нет нигде. И говорили явно не мне.

Edited October 18, 2013 by FaLaRD

[Bassline]

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

[FaLaRD]

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

Я оптимизировал все изображения, фоновая картинка дейтсвительно большая по размеру, но дизайнер предоставил такую и заказчик попросил ничего не менять. Я лишь доработал ее слегка. Графики много, элементов мелких много, оптимизировал спрайтами, css фигурами, и псевдоэлементами.

[advokatua]

На первый взгляд и придраться не к чему (учитывая, что макета мы не видели).

На второй взгляд - бекграунд картинка почти на метр!!! Можно запросто сделать меньше.

Заказчику пробовали объяснить, что ИЕ7 это ископаемое?

График за 2013

Вот бы увидеть его лицо когда он узнает процент конверсии этих 1.8%.

Причем это данные по всему миру, не только для России

1) <h2>INTERTOPS ЗАПУСКАЕТ НОВОЕ ОНЛАЙН-КАЗИНО</h2> - кепс делается через CSS

2) Пустой элемент.

<div class="prev-page">
	 <a href="#"><span></span> Предыдущая</a>
</div>

И тут пачка:

<div class="banners">
<div class="banner-outside fst"></div>
<div class="banner-outside scd"></div>
<div class="banner-outside thrd"></div>
<div class="banner-outside fth"></div>

Понятно, что из-за ИЕ7

3) При добавлении пункта в меню, все падает. Так надо или заказчик сам знает, что там ограничение на 4 пункта?

Причем если убрать пункт, то опять же все едет

PS Тирасполь привет. Лет 6 там не был.

Edited October 18, 2013 by advokatua

[FaLaRD]

d

о_О

[FaLaRD]

На первый взгляд и придраться не к чему (учитывая, что макета мы не видели).

На второй взгляд - бекграунд картинка почти на метр!!! Можно запросто сделать меньше.

Заказчику пробовали объяснить, что ИЕ7 это ископаемое?

График за 2013

Вот бы увидеть его лицо когда он узнает процент конверсии этих 1.8%.

Причем это данные по всему миру, не только для России

1) <h2>INTERTOPS ЗАПУСКАЕТ НОВОЕ ОНЛАЙН-КАЗИНО</h2> - кепс делается через CSS

2) Пустой элемент.

<div class="prev-page">
	 <a href="#"><span></span> Предыдущая</a>
</div>

И тут пачка:

<div class="banners">
<div class="banner-outside fst"></div>
<div class="banner-outside scd"></div>
<div class="banner-outside thrd"></div>
<div class="banner-outside fth"></div>

Понятно, что из-за ИЕ7

3) При добавлении пункта в меню, все падает. Так надо или заказчик сам знает, что там ограничение на 4 пункта?

Причем если убрать пункт, то опять же все едет

PS Тирасполь привет. Лет 6 там не был.

1) По поводу капс лока - в css трансформ прописан. Копировал с дизайна и там так было (не переписывать же мне =) )

2) При добавлении пункта действительн летит, как обычно лучше делать? Я поставил родителю display: table а детям его table-cell, насколько знаю самый гибкий вариант. inline-block неподходит так как оставляет между пунктами 1 пиксель. Посоветуй как сделал бы ты.

3) Пустые элементы пытался как максимум избавиться от них. Где спаны там надо было псевдоэлементы вешать, а вот где пачка? Там не придумал ничего

P.S.: Спасибо за разбор)

[advokatua]

2) При добавлении пункта действительн летит, как обычно лучше делать?

По логике вещей, первое что приходит в голову:

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

Учитывая, что нижнее меню повторяет верхнее, то больше 8 пунктов там быть не должно (по идее).

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

Оверфлоу можно поставить, чтобы не рушилось, и нормик

Ну и остального кода добавить.

Edited October 18, 2013 by advokatua

[ZI DAN]

Насчёт поддержки IE7, я приведу более корректную статистику (мировая статистика для российского сайта несильно применима). Вот статистика с одного моего сайта (у других моих сайтов похожие показатели):

Здесь мы видим, что доля IE в целом далеко не так велика, а доля IE7 вообще не так значительна, чтобы под неё верстать и портить вёрстку в целом.

Что касается тяжести страницы:

Про фон уже сказали. Его вполне можно сделать JPEG'ом, а при должной хитрожопости ещё и отзеркалировать (т. е. размеры картинки уменьшаться вдвое). Многие картинки можно заметить CSS-оформлением (или чуть-чуть упростить и заменить CSS-оформлением), например, кнопки. Другие картинки стоит объединить в спрайт. Также можно посмотреть рекомендации Google.

Edited October 18, 2013 by ZI DAN

[advokatua]

MSIE 4.01 - 1

Это что за?

Типа как в фильме "Пароль Рыба Меч". Какой-то хакер соединился с IBM 5550, пылящегося в подвале универа, соединенного через 12Кбитный модем, и решил полазить по сайтам?

Edited October 18, 2013 by advokatua

[Bassline]

MSIE 4.01 - 1

Это что за?

Типа как в фильме "Пароль Рыба Меч". Какой-то хакер соединился с IBM 5550, пылящегося в подвале универа, соединенного через 12Кбитный модем, и решил полазить по сайтам?

Все версии IE после смерти попадают в интернет. И бродят там, пугая веб-мастеров.