Андрей Булатов
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,  Да, так и сделаю теперь )