Ознакомился со всеми ответами. Спасибо за ваше внимание! Однако, очень хотелось бы увидеть от комментирующих ответы, предложения и замечания по первым двум пунктам.
А так же не вижу смысла останавливаться на проблемах адаптивной верстки. Как было сказано выше ей вообще еще не занимался.
Были комментарии с замечаниями, почему не использовал теги html5 и почему не использовал списки, а ставил вместо них блоки. Ребята, спасибо, я это исправлю. Однако, мне хотелось бы знать, кроме эстетической части если как-то особый смысл заменять блоки, на эти элементы? Вопрос может покажется глупым, но все же
1. Исправим!
2. Ок!
3. Прочитаю обязательно:)
4. Исправим!
5. Вот идея этой самой методологии, чтобы не привязывать стили к определенным элементам. Т.е. на все случаи вешать классы. Насколько я уловил ее суть и старался придерживаться. Отступил кажется только в паре мест
6. Опечаточка
7. Тут стандартное поведение placeholder, решил оставить:)
8. Не увидел:)
@kanonir1886
1. Хорошее замечание! 1 буду фиксить. Второе textarea растянули специально или она так сама встала?
2. В макете картинка на фоне. Если приблизить, то будет заметно, что фон неравномерный, а как-бы "в полоску". Другое серьезное замечание можно было бы сделать по поводу размера этого самого фона. Я хотел сделать его маленьким 10x10 или хотя бы 100x100. Но видимо, я не мастер ФШ и каждый раз у меня заметно терялось качество, поэтому пока оставил так. Но, считаю, что надо это будет исправлять.
3. Хорошее замечание. Однако, я так точно и не знаю влияет ли этот показатель валидности сайта на что-то реально или нет. Ну кроме довольства или не довольства работой самого у себя:)
4. Назначение этого класса было в том, чтобы фоны могли чередоваться. Можно применить и ваше решение, но что мы выигрываем с этого я не очень понимаю
5. Там inline-block чтобы блок не схлопывался, потому что внутри блоки c float. Ваше решение?
6. Интересный момент
7. Если обратить внимание еще и на старые браузеры, то нет))
8. Пожалуй да:)
9. Нет, смотрите документации по этим тегам стандарта html5
10. Именно поэтому не прописываю
11. Пожалуйста, конкретнее? Про то, что где-то после } стоит перенос, а где-то нет. Это неправильно форматирование или имелось ввиду еще что-то?
Rsa97 Что это за устройство? Какое разрешение и какой браузер? Если начали сжимать браузер пока текст не вылез за пределы, ну тут опять же упомяну, что к адаптивной верстке я еще не приступал.
Uwe_Boll По figure, пожалуй, дельное замечание. Со смысловой нагрузкой всех новых тегов html5 я не сильно знаком, поэтому по привычке div-ы. Буду работать над этим.
в clear я ничего преступного не вижу, если можете поясните чем же он хуже чем, цеплять before на элементы?
О списках речь в gallery__preview? Опять же если можно поподробнее, в чем сила, брат?:)
Get в форме по дефолту, я его специально не прописывал. А это ведь только статика. Может быть прикрутится Post, может JavaScript будет эту форму обрабатывать. Не вижу смысла прописывать method="post" на этом этапе
Александр Шемаханов спасибо за ссылку, это мне пригодится! Мобильной версткой еще буду заниматься отдельно. Дизайн будет адаптивным, когда закончу со всеми недоработками desctop версии
Дмитрий Гормаш: Мобильного макета не было, в дальнейшем просто адаптирую этот дизайн. Думаю, его адаптивная верска может заслужить отдельной темы:) Андрей Федоров По поводу 1 и 2, это будет решаться в дальнейшем при помощи media запросов. Вот 3 это интересная вещь. Та же самая верстка локально не уезжает, когда загрузил на гитхаб уехала. Осталось разобраться почему.
Огромное спасибо всем прокомменировавшим! Для меня это очень важно. Честно говоря, даже не ожидал такой оперативной реакции. Буду работать над замечаниями:)
Написано
Войдите на сайт
Чтобы задать вопрос и получить на него квалифицированный ответ.
А так же не вижу смысла останавливаться на проблемах адаптивной верстки. Как было сказано выше ей вообще еще не занимался.
Были комментарии с замечаниями, почему не использовал теги html5 и почему не использовал списки, а ставил вместо них блоки. Ребята, спасибо, я это исправлю. Однако, мне хотелось бы знать, кроме эстетической части если как-то особый смысл заменять блоки, на эти элементы? Вопрос может покажется глупым, но все же