search

DeekHalden / pagespeed

0 stars 0 forks source link

По визуалу(а получилось по коду как всегда) #7

Open preusx opened 7 years ago

preusx commented 7 years ago
  1. http://joxi.ru/KAxXbzXH5N3vA8 - В а блочным элементам не место(по спецификации)(тоесть нужен span). Верстка ломается от браузера к браузеру.
  2. Flex - круто, но либо магия с float никуда не делась и вполне работает, просто более страшненько и не все ее огрехи чистим, либо делаем mobile-first(чего нет, того нет). Но первое все-же предпочтительнее, если у заказчика явно не > IE 11. И даже для него нужно помнить об особенностях: см. Known issues
  3. http://joxi.ru/DmB4zN4tLbwBAP - Картинки не должны деформироваться. Обрезаться - да, можно.
  4. Mobile-first. Так мы говорим IE8 и ниже, что они устарели, и могут потреблять максимум мобильный контент. Еще один плюс - для мобилок легко подключать только им нужные стили. Еще один плюс - эволюционный ход написания кода(от хиленького к перегруженного анимациями). Еще один плюс... Из минусов более высокая сложность(но костылей можно приделать в любой момент если уж сильно прижмет)
  5. Странная структура скриптов. Например googleMap есть в двух местах. Какой из них будет в итоговом бандле? В какой последовательности будет вызван? Какая из initMap все-таки будет в window и какую перезапишут? То-же и к другим относится. Вендоры тоже в двух экземплярах...
preusx commented 7 years ago

http://joxi.ru/krD4jO4tjvLdrp - Не могу свайпать слайдер - сразу показывает lightbox. И ссылку нужно поправить, пусть и на маленькое изображение, но хоть как-то.

DeekHalden commented 7 years ago

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