Jump to content

roman_kr_ru

Newbie
  • Posts

    13
  • Joined

  • Last visited

Posts posted by roman_kr_ru

  1. Не нужно задавать высоту блокам, контент которых будет обновляться динамически. Сейчас если в .info будет больше текста, то он вылезет за пределы блока: http://i61.fastpic.ru/big/2014/0607/e1/34c72531156c55d76b2753ee04b36be1.png Без заданной высоты блок .info "схлопывается", это нормальное поведение, ибо он содержит элементы с float'ом, для того чтобы это не происходило используйте clearfix, например этот: http://nicolasgallagher.com/micro-clearfix-hack/ 

     

    При уменьшении ширины окна браузера, верхняя линия ведет себя не совсем правильно: http://i64.fastpic.ru/big/2014/0607/a4/33e64047699a5f880c1c257ec518ada4.png Для того, чтобы этого не происходило, напишите .line {min-width: 940px;} 

     

    Еще заметил лишний padding 12px у .wrap: http://i64.fastpic.ru/big/2014/0607/c1/8774b549857dc4f3a7351a06434cfec1.png

     

    Возможно, есть смысл добавит ховер эффект у кнопки Read More в слайдере. 

    Кнопки социальных иконок лучше сделать спрайтами. Например, можно использовать онлайн-генератор спрайтов http://ru.spritegen.website-performance.org

     

    Используйте сокращенную запись.

    #thumbs li {
        margin:15px 10px 15px 10px;  //margin: 15px 10px;
    }
    И еще видел, что можно сократить свойство font во многих местах.

     

    Может быть, код 

    .info h1 span, .contact h1 span, .gallary h1 span{
        color:#01a5ca;
    }

    имеет смысл заменить на 

    .first-word {
        color:#01a5ca;
    }
    и в html писать <h1><span class="first-word">Services</span> list</h1>
    Так верстка станет менее "хрупкой", и более независимой от структуры html
     
    reset.css лучше импортировать в style.css, а не подключать через link.
     
    Опечатка в ссылке в index.html: <a href="tim[e]table.html">Timetable</a
     
    Подключайте шрифт через css @import url(http://fonts.googleapis.com/css?family=Asap); cейчас шрифт подключен через тег link, который вы забыли добавить везде, кроме index.html и gall aery.html поэтому на остальных страницах Arial вместо Asap, этого не случилось бы, используй вы @import!
  2. ну как вариант Javascript'ом  смотреть URL , и в зависимости от этого менять стиль body

     

    Для чего JS? Зачем перекладывать на клиент то, что можно сделать на сервере. Лучше посмотреть в сторону php или любого серверного ЯП.

  3. При изменении окна бразуера логотип смещается к центру: не стоило давать ему абсолютное позиционирование. Думаю, стоит обернуть header: 

    <head>  <style type="text/css">    .header {      width: 960px;      margin: auto;    }  </style></head> <div class="header-wrap">  <div class="header">...</div></div>

    Ну для .header-wrap стили 

    .header-wrap {    background: url("../images/header-bg.png") repeat-x;    height: 136px;}

    Еще неясно почему ширина картинки для фона header 200px. Чем руководствовались, когда резали 200px? 

     

    Ну, и самое главное, какого-либо практического значения, эта верстка не имеет. Как заметил npofopr, стоит только сделать длиннее фамилию режиссера и верстка разваливается.

  4. Ludwig Voltman, спасибо вам за  найденные ошибки и за то что помогаете исправлять...очень нужно для начала.Но вот беда,не понимаю, у меня тоже Firefox последний и хедер не пропадает(((что только не клацал,может вы как то по другому просматриваете?

     

    Сделайте ширину окна браузера меньше 650рх и проскролльте вправо - сразу станет видно.

    Можно добавить:

    #header-wrap {min-width: 650px;} и будет нормально.
  5.  

    1. <div class="bx-wrapper" style="max-width: 100%; margin: 0px auto;">

    2. <div class="bx-viewport" style="width: 100%; overflow: hidden; position: relative; height: 300px;">

    3. <ul class="bx-slider" style="width: 415%; position: relative; transition-duration: 0s; transform: translate3d(-325px, 0px, 0px);">

    4. <li class="bx-clone" style="float: left; list-style: none outside none; position: relative; width: 325px;">

    Эти inline-стили можно вынести в css файл, чтобы они не занимали место в html-коде.

    Для слайдера я использовал плагин bx-slider: видимо, эти дивы и стили создаются самим плагином через DOM. Их можно как-то вынести?

  6. 1) не прописана ширина у враппера (body). Если сайт должен тянуться, то необходимо прописать параметры min-width, max-width. Сейчас при изменении ширины окна браузера дизайн разваливается.

    Изменил.

    логотип я бы сделал флоатом

    Попробовал флоат. Все равно наезжает на меню.

    Одни и те же стили написаны и в html (инлайн), и в css

    В html  нет атрибута style, я еще раз проверил даже. Или что Вы имеете ввиду? 

    через blockquote

    Да, согласен, так правильнее.

    Почему кавычки сделаны через рисунок

    Основной шрифт, который я подключил через гугл-фонтс имеет совсем другие кавычки, поэтому сделал через рисунок. Хотя, сейчас понял, что нужно было использовать :before и :after.

     

    однотонная заливка

    Там "шумы" какие-то.

    не gif/png8

    Моя невнимательность.

  7. Спасибо! Учел замечания.

     

     

    1. Где-то id, где-то class , у вас какая-то закономерность

     

     

    Если элемент встречается на странице единожды, использую id, в противном случае  class, так ведь?

     

     

     

    2. Логотип и название сайта, обернуты разными ссылками.

    Название сайта и ссылка внутри h1. Я думаю неправильно будет помещать его в строчный элемент <a></a>. 

×
×
  • 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