Гугловские шрифты оптимальней через @import в style.css сделать
Ну и еще по оптимизации - опустить все скрипты к закрытию body

Не красиво выглядит.

+ Я бы сделал еще что то внизу с ссылками на соц.сети, что бы при наведении что то происходило. Может цвет меняли, или еще что то

Почему? Как я знаю, вариант <link rel="stylesheet"... быстрее нежели @import, в style.css, благодаря параллельной загрузки

Сообщение отредактировал Gerga - 6.2.2016, 23:51

@import отработает один раз и закешируется
link - каждый раз при открытии новой страницы (но не всегда, правда)
Плюс еще один нюанс - это отображение шрифтов до их полной загрузки
В первом варианте будут отображаться чем-то дефолтным, а
во втором - будет "белый" лист до полной загрузки шрифта с гугла....


Стили для :active не прописаны ))

Принимается! Но, , сам link со шрифтами можно разместить до link`а с основными стилями, таким образом мы получим шрифты на ранней стадии прорисовки

pixmart, подключайте скрипты перед </body>, первый div (<div class="warning" >) физически также разместить в самый низ страницы. Используйте article и section там, где можно их использовать. Используйте nav. Нужно только 1 заголовок первого уровня использовать на странице.

Сообщение отредактировал Gerga - 7.2.2016, 0:52

1. Этого лучше избегать, если имитируете мобильник и т.п., особенно если присутствиет autoplay (сугубо личное мнение). И вообще, советую юзать owlcarousel.


2. Для таких моментов можно использовать :first-child. Нет, я не говорю, что это ошибка и/или плохо, ну просто, на мой взгляд, нужно приучать себя к современным стандартам. И, как правило, кнопки типа скачать и читать далее являются обычными ссылками, button в данном случае не уместен.


3. Упрощаем код, да и не очень валидно. Приучайте себя не создавать тысячу вложений, если ситуация вполне позволяет этого избежать.




4. Будьте внимательней


Как следствие:



5. В отдельный файл


6. Этого делать не обязательно для header, нет смысла задавать ему класс, он не повторяется на странице. Ну и почему бы не использовать помимо header и footer другие элементы HTML5, если уж взялись за него (например section и т.д.)


7. br здесь лишний, достаточно задать для конкретного span - display:block


Ну и на этом я осмотр завершил. Макет не видел.

falken, весьма поучительно, огромное СПАСИБО. Будем учиться.

Лично я не "доверяю" ни гуглу, ни кому другому (из сервисов)
И в случае если вынужден юзать нестандартные шрифты, я их загружаю на свой хостинг(впс и т.п.) , а там уже @font-face

mmkulikov, подгружаю шрифты через Web Font Loader от Google. Все нравится, хороший контроль процесса загрузки. Позволяет использовать множество шрифтов, подгружая их последовательно или асинхронно, на слабых соединениях так же показывает дефолтный шрифт до загрузки. Добавляет тест-классы в html, в процессе загрузки и после ее окончания или ошибки загрузку, что позволяет среагировать в css на ошибки.

Сообщение отредактировал roamn - 7.2.2016, 16:53