Андрей Булатов
13 повідомлень
#11 років тому
Здравствуйте друзья! 
Я верстальщик самоучка, учился по книгам, сверстал пару сайтов для получения хоть какой-то практики. 
Выставляю на Вашу беспощадную критику свою верстку, можете забрасывать меня камнями. 
Владислав Р.
175 повідомлень
#11 років тому
Выкладывай верстку на бесплатном (или платном) хостинге, если хочешь что бы её смотрели. Архив никто качать не будет.
Андрей Булатов
13 повідомлень
#11 років тому
Vlad03,  Спасибо за совет! 
Вот ссылка по которой можно перейти на сайт 
Владислав Р.
175 повідомлень
#11 років тому
Проверь на валидаторе, там есть ошибки.
Кнопка "Read More", все таки, кнопка или ссылка?
Андрей Булатов
13 повідомлень
#11 років тому
Vlad03,  ошибку исправил. 
"Read More" - кнопка. Я так понимаю что у кнопок не должно быть подчеркивание? 
в меню футера не получается применить first-child что бы убрать отступ слева. Применяется к всем ссылкам меню. Подскажите в чем проблема. 
Антон В.
1807 повідомлень
#11 років тому
Цитата (andrehtml):
в меню футера не получается применить first-child что бы убрать отступ слева
Вы покажите, как вы его прописываете для начала....сейчас там ничего не прописано.
А так в целом  -  вобщем-то очень даже ничего вёрстка.
Андрей Булатов
13 повідомлень
#11 років тому
nehovaysyatopol, Спасибо за оценку! 

Для меню  в футере сейчас прописал 


.copy-menu ul li a:first-child {
padding:0 8px 0 0;}
Сергей Л.
616 повідомлень
#11 років тому
.copy-menu ul li a:first-child
вот так надо
.copy-menu ul li:first-child a
Андрей Булатов
13 повідомлень
#11 років тому
WeGeeks, Сработало, спасибо!
Виктор Т.
1 повідомлення
#11 років тому
Хм, я не понимаю - а для чего на страничке подключен SyntaxHighlighter? Да и Modernizr вроде бы не используется, можно было бы обойтись html5shiv для совместимости с ослами.
Андрей Булатов
13 повідомлень
#11 років тому
zenw, Без SyntaxHiglighter не работает слайдер так как должен работать. 
По Modernizer тут согласен, можно убирать, он попал в верстку вместе с слайдером. 
zenw, ну а так, в целом что скажете про верстку? 
Татьяна К.
51 повідомлення
#11 років тому
Из критики:
1) выбран не самый лучший плагин для слайдера (слишком много скриптов для одной простой листалки)
2) отсутствует кроссбраузерность как таковая, т.е. вёрстка уже не рабочая (!!! самое важное)
3) скрипты для вёрстки должны всегда быть убраны в файлы, а не валяться в html (исключение - яндекс.карты разве что)
Ну а так для начала хорошо =) У местных фрилансеров периодически вижу код в разы хуже
Андрей Булатов
13 повідомлень
#11 років тому
Mideliya, Спасибо! =))
По кроссбраузерности Вы про  IE8?
Владислав Р.
175 повідомлень
#11 років тому
Молодой человек, будь, я админом сего сервиса, я б банил вас "по IP" забанил, и так бы вам и надо. Вас в гугле забанили?
Андрей Булатов
13 повідомлень
#11 років тому
Vlad03,  пока нет, но намек я понял. 
Владислав Р.
175 повідомлень
#11 років тому
Вы ж, элементарные вопросы задаёте.
Если, что не понятно, можно в личку, ну и тут конечно, без обид.)
Цитата (nehovaysyatopol):
в общем-то очень даже ничего вёрстка
Тут поддержу, дальше всё это мелочи,  бери проекты и не заморачивайся, там по ходу выплывет, что надо и не надо.
З.Ы. У местных "верстал" есть говно-код в разы хуже вашего.
Андрей Булатов
13 повідомлень
#11 років тому
Vlad03,  та я знаю что элементарные, просто у меня все время такое впечатление что где то что то упустил или что то не то сделал)) поэтому и переспрашиваю все время,  ну, теперь точно подозреваю что кросс по IE.)
Само собой что без обид))
Спасибо за критику, очень благодарен)
Владислав Р.
175 повідомлень
#11 років тому
Цитата (andrehtml):
теперь точно подозреваю что кросс по IE.)
Так ставите виртуалку и "любимый" IE и всех делов-то.
Андрей Булатов
13 повідомлень
#11 років тому
Vlad03,  Да, так и сделаю теперь )