Александр Ф.
22 сообщения
#11 лет назад
Владислав Р.
175 сообщений
#11 лет назад
Слайдер угомоните, а то голова кружится.
Евгений Еременко
131 сообщение
#11 лет назад


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

upd:

и сглаживание шрифтов сделайте, а то в хроме ужасненько
Александр Ф.
22 сообщения
#11 лет назад
Vlad03, один раз клацнуть на стрелочку и он остановиться
Александр Ф.
22 сообщения
#11 лет назад
eremenko-evgeniy, граница фонового изображения. Оно же не бесконечной ширины.
Владислав Р.
175 сообщений
#11 лет назад
Цитата ("DrGonzo"):
Vlad03, один раз клацнуть на стрелочку и он остановиться

можно время переключения больше поставить и жать ничего не надо будет
Александр Ф.
22 сообщения
#11 лет назад
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
Александр Ф.
22 сообщения
#11 лет назад
parkgraphic, а как же количество запросов серверу?
Антон Б.
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 сообщения
#11 лет назад
Мелочей много, но а в целом - неплохо
Единственная проблемка, свернув окно, и развернув обратно, получилось вот это:
Александр Ф.
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 создать просто несколько параграфов с ссылками, списков и таблиц, будет очень некрасиво...
Зачем это делать?