Оцените вёрстку
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 создать просто несколько параграфов с ссылками, списков и таблиц, будет очень некрасиво...
Зачем это делать?







