Jump to content

Оцените пожалуйста верстку (адаптивную)


Нарек
 Share

Recommended Posts

Хотелось бы узнать мнение более опытных, я собираю портфолио, стоит ли такую работу туда выкладывать?

Какие задачи я ставил для себя:

1. Адаптивность - минимальная ширина 320px, IE 8 и выше

2. Кроссбраузерность - IE8 и выше

3. Валидность кода - постоянная моя забота

4. Логичность и лаконичность кода

Pixel Perfect не удалось, есть минимальные различия в некоторых деталях (~5-8px), из за слабой управляемости сетки. Сетку взял из Bootstrap (Только сетку).

 

Прошу сказать что не удалось, буду очень признателен.

 

http://www.nt-work.bl.ee/

 

 

upd. Firefox не понимает подключенный шрифт, пытаюсь найти решение.

Edited by Нарек
Link to comment
Share on other sites

Для футера не нужен nav http://www.whatwg.org/specs/web-apps/current-work/multipage/sections.html#the-nav-element. Для дат есть time, и у вас transition срабатывает только в одну сторону.

 

upd: Думаю в эмульяторе ff без скролла, потому что там нет вертикального скролла-бара справа, который есть в хроме

Edited by xzarxzes
Link to comment
Share on other sites

334px уже скролл появился

Исправил

Для футера не нужен nav

Я читал недавно содержимое этой ссылки, там говориться, что не все группы ссылок нужно обернуть в nav, но у меня в футере полноценное меню а не сокращенная группа. Ну был бы там только home, blog, и копирайт, то не обернул бы. Если я не правильно понял суть содержимого ссылки объясните пожалуйста.

Для дат есть time, и у вас transition срабатывает только в одну сторону.
 исправил.

 

Спасибо ребята за отклик. Жду еще критики) 

Link to comment
Share on other sites

Впринципе я на 100% не уверен на счет такой навигации, там сказанно что самого футера достаточно и что он вроде как сам по себе подразумивает навигацию, там так же говорится что это всего лишь необязательно делать nav в футере, так что это не критично если она у вас там будет. Но мне кажется что футер не самое подходящее место для главной навигации, даже если она дублирует навигацию сверху, которая главная, как в данном случае, думаю это излишне ставить там nav, можно было обойтись обычнм ul. В футре находится различная доп. информация - контакты, копирайт итд, а главная навигация, она же главная, и должна находится "на более видном месте". Это лишь мои соображения, и если я неправильно понял, то пусть меня кто нибудь поправит. 

 

upd: вот можно здесь еще почитать http://css-live.ru/articles/semanticheskaya-navigaciya-pri-pomoshhi-elementa-nav.html, также можно представить что кто то, кто использует скринридер зашел на ваш сайт, насколько мне известно скринридер будет делать акцент на главной навигации, в таком случае получится что он дважды будет обращать внимание на одну и ту же навигацию, сверху и в футере. Это также только лишь домыслы, я не особо представляю как пользуются скринридером.

Edited by xzarxzes
Link to comment
Share on other sites

@xzarxzes,  Спасибо за подробное описание. 

 

Вот что заметил, эту строку FF не понимает.

font: 600 16px/68px 'Open Sans' Arial, Verdana, sans-serif;

Для него нужно написать все отдельно, font-weight: 600; line-height: 68px; 

Почему это так, строка неправильная или ФФ чудит?

 

да и подключенный шрифт не понимает он...

<link href='http://fonts.googleapis.com/css?family=Open+Sans:400,600,700' rel='stylesheet' type='text/css'>
Edited by Нарек
Link to comment
Share on other sites

 

@xzarxzes,  Спасибо за подробное описание. 

 

Вот что заметил, эту строку FF не понимает.

font: 600 16px/68px 'Open Sans' Arial, Verdana, sans-serif;

Для него нужно написать все отдельно, font-weight: 600; line-height: 68px; 

Почему это так, строка неправильная или ФФ чудит?

 

да и подключенный шрифт не понимает он...

<link href='http://fonts.googleapis.com/css?family=Open+Sans:400,600,700' rel='stylesheet' type='text/css'>

У вас запятой нету после 'Open Sans', наверно поэтому он не понимает

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