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