Я по специальности не программист, но недавно поняла что программирование это именно то чем я хочу заниматься. Итак изучив некоторые его инструменты я сама создала сайт для знакомой художницы. Оцените, пожалуйста, его. Так как для меня очень важен взгляд со стороны.

http://dasha-ivashchenko.com

Оценить:|2
  
Комментировать
Раздел: Обучение    
Просмотров: 1291
Комментариев (19)
  1. dikucher
    Дмитрий [dikucher]
    17.09.2013 в 15:12

    а что именно вы здесь делали ? дизайн, верстку, посадку на движок. или все это с написанием движка с нуля ?

    • Ирина Шкурат [irinashkurat]
      17.09.2013 в 15:18

      Всё полностью создавала сама. Сначала придумала дизайн сайта. Так как я пока в этом не специалист, то дизайн простенький получился. Потом сверстала страницы используя HTML и CSS, а потом уже для того что-бы всё динамически работало использовала PHP и MySQL. На некоторых страницах я также использовала JS, но так как я в нём не очень пока разбираюсь, то находила в интернете подходящий мне код и меняла под свои потребности.

  2. dikucher
    Дмитрий [dikucher]
    17.09.2013 в 15:19

    имхо касательно того, что вижу
    1. не нравиться как прикреплен блок контента к меню, я бы сделал отступ как перед футером
    2. у вас когда находишься на странице, соответствующее меню подсвечивается розовым, но не подсвечивается вкладка «ваши вопросы»
    3. откуда этот розовый цвет ???? как по мне, он здесь вообще не к месту.
    4. сделал бы логотип кликабельным с переходом на главную, ( продублировал бы функции «Главная»), может вы посчитали не нужным, при наличии кнопки «Главная», но мне кажется что кликабельный логотип — это как правило по умолчанию
    5. + фавикон

    Повторюсь что это только — имхо

    • Ирина Шкурат [irinashkurat]
      17.09.2013 в 15:26

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

  3. tipois
    Ринат Вольнов [tipois]
    17.09.2013 в 17:43

    Много чего можно написать (про дизайн, верстку). Остановлюсь лишь на том, что дико цепляет глаз, так это курсивный шрифт по всему сайту, дико снижает читабельность. Курсив нужен лишь для «интонации». Вообще советую почитать для начала Стива Круга `Не заставляйте меня думать!`, книга довольно старая(она вам даст опору, от которой можно оттолкнуться), но актуальна и сейчас. Успехов!

  4. tipois
    Ринат Вольнов [tipois]
    17.09.2013 в 17:48

    И да, думаю вам нужно двигаться в сторону блога.

  5. dikucher
    Дмитрий [dikucher]
    17.09.2013 в 18:07

    Также заметил что для верстки вы не уместно использовали таблицы. Таблицы используйте только для отображения таблиц.
    зачем
    на странице «услуги» ? если вам надо было ниже опустить надпись «Услуги», то достаточно было добавить margin к элементу h1

    • tipois
      Ринат Вольнов [tipois]
      17.09.2013 в 18:32

      Синематически да «таблицы для табличных значений» (сам приверженец этого), но бывает что табличная конструкция куда «легче» блочной, так что тут от случая зависит, конечно же нельзя злоупотреблять. Это можно сравнить с !important в верстке его нужно избегать, но местами он необходим(например определенные плагины для браузеров).

      • Ирина Шкурат [irinashkurat]
        17.09.2013 в 18:45

        Я когда делала страницу услуги, то у меня возникли проблемы с позиционированием divов, поэтому и сделала через таблицу. Хотя понимаю что лучше блочно верстать

        • dikucher
          Дмитрий [dikucher]
          17.09.2013 в 19:15

          у вас наверно возникли сложности с тем, чтобы спозиционировать текст вертикально по центру
          как вариант, следующий код без использовании таблицы ( правда с её имитацией )
          html

          <div class="service">
               <p><a href="">Портрет на фото</a></p>
               <img alt="" src="img/foto1.jpg">
          </div>

          CSS

          p {
              float: left;
          }
          a {
              display: table-cell;
              vertical-align: middle;
              height: 153px;   /*высота вашей картинки */
              width: 190px;    /* ширина для текста */
          }
        • tipois
          Ринат Вольнов [tipois]
          17.09.2013 в 18:58

          Если честно, то там не рационально используется пространство. И впрочем над структурой тоже стоит подумать.

  6. tipois
    Ринат Вольнов [tipois]
    17.09.2013 в 19:03

    Совет на будущее, если хотите быть лучшим, развивайтесь в каком-то одном направлении, будь то верстка, программирование, дизайн ect.

    • dikucher
      Дмитрий [dikucher]
      17.09.2013 в 19:20

      я бы выразился немного по другому.
      Если вы хотите быть лучшим в верстке, то развивайте только верстку.

      Но зачастую это не то что вам будет нужно. Выберите основное направление и уделяйте ему 80-90% времени и сил, но при этом паралельно изучайте смежные отрасли.

  7. LeechAXSS
    17.09.2013 в 19:24

    Все очень красиво получилось, ставлю5! Но чуть-чуть допилить и пусть живет людей радует, женских рук творенье))
    Я бы еще форму обратной связи поправил, кнопочку переделал в button, выровнил ширину инпутов, оформления им добавил, но это не обязательно, конечно я бы так сделал))

    • dikucher
      Дмитрий [dikucher]
      17.09.2013 в 19:45

      при верстке формы тоже таблица (

      <form class="feedback">
      <div>
          <label for="name">textdfsdfsd</label>
          <input id="name" type="text">
      </div>
      <div>
          <label for="email">text</label>
          <input id="email" type="text">
      </div>
      <div>
          <label class="style1" for="message"><span>text text text</span></label>
          <textarea id="message"></textarea>
      </div>
      </form>

      CSS

      .feedback label {
          width: 100px;
          display: inline-block;
      }
      .feedback input, .feedback textarea {
          width: 200px;
      }
      .style1 span {
          display: table-cell;
          vertical-align: middle;
          height: 54px;
      }
      .feedback textarea {
          display: inline;
          height: 54px;
      }
      • dikucher
        Дмитрий [dikucher]
        17.09.2013 в 19:51

        ах еще и кнопка, html вставить перед закрывающимся тегом FORM

            <div>
                <input id="button" type="submit" value="отправить">
            </div>
        #button {
            width: 100px;
            margin: 0 0 0 100px;
        

        Числовые значения надо будет подправить под ваш дизайн

  8. dimka
    Dimka Klusevich [dimka]
    18.09.2013 в 11:15

    Что касается дизайна, я бы попробовал бы немного стилизовать слайдер, навигация слайдера выпадает из общего стиля… Как было сказано выше, курсив общего контента, это не очень хорошее решение, вообще не стандартные шрифтовые решения, это очень специфический момент в веб дизайне, с этим нужно быть аккуратней… Да с типографикой на всех страничках нужно поработать. Я бы упразднил копку «на главную», и как было сказано выше, сделал бы кликабельным логотип, с ссылкой на главную страницу, конечно, что кроме самой главной страницы. Блок меню имеет прямые углы, а остальные блоки скругленные, нужно выбрать что то одно… Не забывать про ховеры, в меню портфолио их нет…

    не очень понятно зачем в использовали здесь span
    <li><a href="portfolio.php"><span>ПОРТФОЛИО</span></a></li>
    
    вот такая конструкция будет более правильной
    <li><a href="portfolio.php">ПОРТФОЛИО</a></li>
    

    В верстке советовал бы лучше использовать классы, вместо id(их лучше оставить для JS)….
    По back-end части сказать, не чего не могу. потому что занимаюсь в основном версткой и front-end разработкой… Если возникнут вопросы обращайтесь в личку…
    p.s. для первого раза совсем не дурно…. )

  9. moroz69off
    Юрий Морозов [moroz69off]
    19.09.2013 в 19:03

    По необходимости оформляеться в рамку (под стекло). — было,

    По необходимости работа оформляется в рамку (под стекло). — надо.

    Хiба на мовi…

    Про курсив тут уже говорили.

Оставить комментарий