Оцените вёрстку
131 повідомлення
#11 років тому

DrGonzo, это что за "огрызки" по бокам слайдера?

upd:
и сглаживание шрифтов сделайте, а то в хроме ужасненько
22 повідомлення
#11 років тому
eremenko-evgeniy, граница фонового изображения. Оно же не бесконечной ширины.
175 повідомлень
#11 років тому
Цитата ("DrGonzo"):Vlad03, один раз клацнуть на стрелочку и он остановиться
можно время переключения больше поставить и жать ничего не надо будет
616 повідомлень
#11 років тому
Слишком громоздкий код, можно было в половину меньше написать и получить тоже самое. Меню обычно делают маркированным списком. Простейший макет, особо оценивать нечего.
22 повідомлення
#11 років тому
art-apple, маркированным списком не делал специально - лишние тэги.Про громоздкость можно подробнее?
Как уменшить?
UPD:
Вы имели ввиду HTML-код. Я подумал про CSS.
Вопрос снят.
1132 повідомлення
#11 років тому
Треть тегов, действительно, не нужна.Меню списком однозначно.
Уберите выделение внутри слайдера. Многочисленные клики по стрелкам выделяют весь слайд.
Слайд некрасиво уезжает под стрелки.
В целом чистенько. Верстаете лучше доброй половины верстальщиков.
22 повідомлення
#11 років тому
parkgraphic, дело в том, что была задача обеспечить работу в IE8. А он не поддерживает multiple backgrounds и другие вещи. Приходилось добавлять тэги.С замечаниями согласен, спасибо.
1132 повідомлення
#11 років тому
DrGonzo, и я бы все иконки сделал обычным img, а не backgroundИбо в большинстве случае верстка натягивается на CMS и было бы удобно заменять иконки прям через админку, а не копаясь в css
1132 повідомлення
#11 років тому
Не в этом случае))
39 повідомлень
#11 років тому
DrGonzo, я бы порекомендовал пересмотреть систему сокращений названия класса.Понять, что такое pmsbswsb-text, pmfb-text практически невозможно.

22 повідомлення
#11 років тому
pettson, да, это её недостаток. Но новую систему пока не придумал. А БЭМ мне не нравиться. Вообщем, в процессе...
51 повідомлення
#11 років тому
Хоть и прошло 3 дня, некоторые вещи не назвали: отсутствие html5 в принципе при указанном doctype, но это скорее просто странно. Кстати навигацию лучше было бы делать nav'ом, а не обычным div'ом. Ну или списком на худой конец.
Из очень серьёзного: h1 в слайдере, повторяющийся несколько раз. На странице должен быть только один h1, но по-хорошему расстановку всех заголовков надо обсуждать с сеошником, который будет работать с сайтом.
Событие js в самом html - тоже не есть хорошо (onclick="mySwipe.prev()" у стрелок слайдера)
Шрифты лучше подключать в css, js - выносить в отдельный файл. Стили должны быть объединены в одном *.css.
Также не прописаны дефолтные стили (т.е. если в main создать просто несколько параграфов с ссылками, списков и таблиц, будет очень некрасиво).
2868 повідомлень
#11 років тому
Цитата ("Mideliya"):Из очень серьёзного: h1 в слайдере, повторяющийся несколько раз. На странице должен быть только один h1, но по-хорошему расстановку всех заголовков надо обсуждать с сеошником, который будет работать с сайтом.
h1 в слайдере вовсе не должно быть, так как тег создан для указания названия страницы.
182 повідомлення
22 повідомлення
#11 років тому
Цитата ("Mideliya"):Хоть и прошло 3 дня, некоторые вещи не назвали:
отсутствие html5 в принципе при указанном doctype, но это скорее просто странно. Кстати навигацию лучше было бы делать nav'ом, а не обычным div'ом. Ну или списком на худой конец.
Из очень серьёзного: h1 в слайдере, повторяющийся несколько раз. На странице должен быть только один h1, но по-хорошему расстановку всех заголовков надо обсуждать с сеошником, который будет работать с сайтом.
Событие js в самом html - тоже не есть хорошо (onclick="mySwipe.prev()" у стрелок слайдера)
Шрифты лучше подключать в css, js - выносить в отдельный файл. Стили должны быть объединены в одном *.css.
Также не прописаны дефолтные стили (т.е. если в main создать просто несколько параграфов с ссылками, списков и таблиц, будет очень некрасиво).
Отсутствие HTML5...
Сделано специально. Потому что IE8 не понимает все эти новые тэги.
h1...
Cогласен. Учту.
Событие JS...
Я в JS вообще чайник. Первый раз использовал. Где-то читал что лучше JS-код размещать в конце HTML-кода. Так и сделал.
Шрифты лучше подключать в css...
Так и сделано.
...если в main создать просто несколько параграфов с ссылками, списков и таблиц, будет очень некрасиво...
Зачем это делать?







