Jump to content

Нужна критика верстки


Stright
 Share

Recommended Posts

Доброе утро всем! Посмотрите, пожалуйста, верстку макета. Его уже выкладывал один пользователь на обсуждение, и я решил взять его в качестве первой верстки. Верстал под ei9+. При верстке заметил, что firefox некорректно отображает шрифты, но при обновлении на последнюю версию браузера все стало отображаться нормально (может быть неправильно подключены шрифты?). Еще на домашнем и рабочем компьютере шрифты отображаются как надо, но при просмотре страницы на компьютере друга (ie9  и chrome) - опять некорректное отображение шрифтов. С чем это может быть связано?

Верстка - http://stright.url.ph/

Макет - https://yadi.sk/d/KbwvqaN0bUaom

Код - https://yadi.sk/d/60nXFePGbUaq8

Link to comment
Share on other sites

Спасибо за просмотр ) Обнаружил одну проблемку в ie9 - абсолютно спозиционированная стрелка (section>div.content_wrap>a>span) отображается немного не так - выше на 3px. Как это лучше исправить? Написать отдельный файл стилей для этого элемента и подключить через условный комментарий для ie9? Или написать стили для этого элемента прямо в условном комментарии?

Edited by Stright
Link to comment
Share on other sites

Обнаружил одну проблемку в ie9 - абсолютно спозиционированная стрелка (section>div.content_wrap>a>span) отображается немного не так - выше на 3px.

У меня подозрение, что в ИЕ9 не подключился шрифт lato, и там отображается дефолтный serif.

Link to comment
Share on other sites

а почему вы практически не используете классы? section a:first-childб section a:last-child - ну это не серьезно. вставляем ссылки в текст получаем чудо эффект http://take.ms/h9u1V

ну и нельзя задавать стили просто на теги. вы глобально задаете стили на header, section, main, article, footer - это очень плохо. тегов header и footer легко может быть несколько на странице, про section, main, article я вообще молчу. у вас уже начал валится сайт когда я вставил две ссылки при том что тут одна малюсенькая страничка, что же будет когда будет большой сайт? обязательно задавайте классы! не бойтесь их, хотя нужно тоже меру знать. с другой стороны если взять любой фреймворк типа бустрапа - так просто миллион классов, и ничего, никто не умер. в вашем случае будет достаточно просто сюда http://take.ms/7Lpbl класс, к примеру, .btn-more или .link-more.

идем дальше http://take.ms/K9azt если вставить картинку большего размера то сайт повалился (обязательно кто-то влепит такую картинку, даже не сомневайтесь). тут я как бы поступил http://codepen.io/Z1gnet/pen/dgxIs

ну и эта конструкция меня напрягает http://take.ms/6LJat . во первых не хорошо логотип в H1 сувать, а во вторых, опять же, давайте классы или айдишники. было бы просто .logo span.

  • Like 1
Link to comment
Share on other sites

Еще на домашнем и рабочем компьютере шрифты отображаются как надо, но при просмотре страницы на компьютере друга (ie9  и chrome) - опять некорректное отображение шрифтов.

 

Оказалось и на работе, и дома эти шрифты были установлены :facepalmxd:

 

 

У меня подозрение, что в ИЕ9 не подключился шрифт lato, и там отображается дефолтный serif.

 

Спасибо, дело оказалось действительно в этом, после того, как подключил шрифты по-нормальному, проблема исчезла  :)  

 

 

что же будет когда будет большой сайт? обязательно задавайте классы!

 

 

Как раз и не стал задавать классы, потому как страничка простая, хотя ссылкам это действительно было необходимо, думал об этом, когда смотрел верстку в IE8, но верстал под IE9+, поэтому оставил все как есть. А вот про то, что, если добавить еще ссылки, то последний элемент уже станет не последним, как-то не подумал  :)

Про картинки не сообразил, хорошее замечание, спасибо, лого тоже поправил.

 

Большое спасибо всем отписавшимся!

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy