• Оцените верстку?

    @oleg-s Автор вопроса
    Ознакомился со всеми ответами. Спасибо за ваше внимание! Однако, очень хотелось бы увидеть от комментирующих ответы, предложения и замечания по первым двум пунктам.
    А так же не вижу смысла останавливаться на проблемах адаптивной верстки. Как было сказано выше ей вообще еще не занимался.
    Были комментарии с замечаниями, почему не использовал теги html5 и почему не использовал списки, а ставил вместо них блоки. Ребята, спасибо, я это исправлю. Однако, мне хотелось бы знать, кроме эстетической части если как-то особый смысл заменять блоки, на эти элементы? Вопрос может покажется глупым, но все же
  • Оцените верстку?

    @oleg-s Автор вопроса
    1. Исправим!
    2. Ок!
    3. Прочитаю обязательно:)
    4. Исправим!
    5. Вот идея этой самой методологии, чтобы не привязывать стили к определенным элементам. Т.е. на все случаи вешать классы. Насколько я уловил ее суть и старался придерживаться. Отступил кажется только в паре мест
    6. Опечаточка
    7. Тут стандартное поведение placeholder, решил оставить:)
    8. Не увидел:)

    Спасибо за ваш комментарий!
  • Оцените верстку?

    @oleg-s Автор вопроса
    @kanonir1886
    1. Хорошее замечание! 1 буду фиксить. Второе textarea растянули специально или она так сама встала?
    2. В макете картинка на фоне. Если приблизить, то будет заметно, что фон неравномерный, а как-бы "в полоску". Другое серьезное замечание можно было бы сделать по поводу размера этого самого фона. Я хотел сделать его маленьким 10x10 или хотя бы 100x100. Но видимо, я не мастер ФШ и каждый раз у меня заметно терялось качество, поэтому пока оставил так. Но, считаю, что надо это будет исправлять.
    3. Хорошее замечание. Однако, я так точно и не знаю влияет ли этот показатель валидности сайта на что-то реально или нет. Ну кроме довольства или не довольства работой самого у себя:)
    4. Назначение этого класса было в том, чтобы фоны могли чередоваться. Можно применить и ваше решение, но что мы выигрываем с этого я не очень понимаю
    5. Там inline-block чтобы блок не схлопывался, потому что внутри блоки c float. Ваше решение?
    6. Интересный момент
    7. Если обратить внимание еще и на старые браузеры, то нет))
    8. Пожалуй да:)
    9. Нет, смотрите документации по этим тегам стандарта html5
    10. Именно поэтому не прописываю
    11. Пожалуйста, конкретнее? Про то, что где-то после } стоит перенос, а где-то нет. Это неправильно форматирование или имелось ввиду еще что-то?

    Спасибо за подробный комментарий!
  • Оцените верстку?

    @oleg-s Автор вопроса
    Rsa97 Что это за устройство? Какое разрешение и какой браузер? Если начали сжимать браузер пока текст не вылез за пределы, ну тут опять же упомяну, что к адаптивной верстке я еще не приступал.
  • Оцените верстку?

    @oleg-s Автор вопроса
    Uwe_Boll По figure, пожалуй, дельное замечание. Со смысловой нагрузкой всех новых тегов html5 я не сильно знаком, поэтому по привычке div-ы. Буду работать над этим.
    в clear я ничего преступного не вижу, если можете поясните чем же он хуже чем, цеплять before на элементы?
    О списках речь в gallery__preview? Опять же если можно поподробнее, в чем сила, брат?:)
    Get в форме по дефолту, я его специально не прописывал. А это ведь только статика. Может быть прикрутится Post, может JavaScript будет эту форму обрабатывать. Не вижу смысла прописывать method="post" на этом этапе
  • Оцените верстку?

    @oleg-s Автор вопроса
    Александр Шемаханов спасибо за ссылку, это мне пригодится! Мобильной версткой еще буду заниматься отдельно. Дизайн будет адаптивным, когда закончу со всеми недоработками desctop версии
  • Оцените верстку?

    @oleg-s Автор вопроса
    Дмитрий Гормаш: Мобильного макета не было, в дальнейшем просто адаптирую этот дизайн. Думаю, его адаптивная верска может заслужить отдельной темы:)
    Андрей Федоров По поводу 1 и 2, это будет решаться в дальнейшем при помощи media запросов. Вот 3 это интересная вещь. Та же самая верстка локально не уезжает, когда загрузил на гитхаб уехала. Осталось разобраться почему.
  • Оцените верстку?

    @oleg-s Автор вопроса
    Огромное спасибо всем прокомменировавшим! Для меня это очень важно. Честно говоря, даже не ожидал такой оперативной реакции. Буду работать над замечаниями:)