Mirage

Странно центрируйте элементы под слайдером, не проще ли им всем сделать float: left?

В IE8 скрипты не работают.

Лучше писать на jquery,

ну а так нормально

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

1. не использовать ID

2. #topMenu li сделать без картинок

3. Для слайдера использовать любое гтофое решение (например jCarousel) вместо непонятно самописа.

4. не использовать верхний регистр в именах классов

.functionsTitle -- не правильно

.functions-title -- правильно, хотя назвать данный элемент можно было бы более юзер френдли.

5. #functions .content -- сделать без картинок

6. #topSwitch лучше переделать на базе jQuery UI Tabs

7. #search -- скорее всего контент надо было положить в <fieldset>

8. #popular - это должно быть список ul>li

9. #popular .post и #content .post -- ты назвал одинаково (.post) разные сущности, что не правильно. каждая сущность должна иметь своё имея и набор свойств независимо от других.

10. .title .description -- зачем на <span>вешать display: block; ?? не логичнее было использовать <div>

11. содержимое блока: .postMessage сделано не правильно. Тут должно быть разделение. Отдельно ссылка, в которой картинка. Отдельно <div> для текста. Текст скорее всего в будущем будет лежать в <p>, так как его будут наполнять в редакторе.

12. #botWrap - некорректное именование. Не следует сокращать термины. Лучше написать .bottom-wrapper что будет более понятно в будущем и тебе и другому разработчику который, возможно, будет работать над проектом.

13. #botMenu -- вся структура заведомо имеет ошибку. А что случится если завтра вмето картинок справа добавится еще 3 колонки со списком? Кастомеру надо будет лезать в разметку и её менять, что ли? Вообще такая индексация (.first, .second, etc) не требуется в 99,9% случаев а вероятность того что тебе она понадобится в ближайшие несколько лет близится к нулю. Перестань так делать. Научись предполагать что будет происходить с кодом если завтра контент будет изменен. Тут главное что бы была гибкость и простота настолько насколько это возможно.

Отлично! Спасибо за подробные замечания и за время, уделенное на это. Буду исправляться. А насчет названий классов, CamelCase(или это неуместно?).

Написал заново js, теперь вроде бы нормально.

Обновил тему.

Прошу Вас оценить верстку макета(найденного на просторах интернета) и самое главное указать на ошибки.

— Верстка

— Макет

Я начинающий верстальщик. Тени реализовать получилось не все. Обратите внимание на то, что я не использовал плюшки html5 и css3 намеренно(такую задачу я себе поставил), так что прошу не писать об этом.

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

Буду благодарен за Ваши замечания и советы. Спасибо всем, кто посетил эту тему.