Р
Но это не критично, если что. Просто это не адаптив для экранов выше 1440
Size: a a a
2019 July 28
Р
Ну это не адаптив.
Р
Оууу, у тебя эта кнопка - не кнопка, а картинка - это халтура.
Р
Вот понадобится ее анимировать, чтобы стрелочка внутри двигалась - тогда переделывать? Это ты зря.
Р
Без комментариев
Р
Еще почитай про семантику - у тебя куча h1 на странице, а может быть только один.
Р
Классы хорошо прописаны, со стороны вроде все прилично.
Р
Еще мелкие моменты по грамотности\внимательности.
У тебя кнопка events ведет к разделу ivents. Не всегда есть контент-менеджеры, которые это заметят и поправят. Стоит обращать внимание.
У тебя кнопка events ведет к разделу ivents. Не всегда есть контент-менеджеры, которые это заметят и поправят. Стоит обращать внимание.
Р
Еще br стоит использовать только в тех случаях, когда ты на 100% уверен, что это нельзя будет менять через админку. Т.е. лучше вообще не использовать.
Р
Но хорошая работа, у меня говнокод и того хуже.
E
Мне слайдер на мини и самому не очень понравился, да и многие нюансы, тот же ивент , просто не заморочился, а вот по якорю и контейнеру, я как то не подумал. А и в формах, хотел уточнить, выходит их только с помощью js можно сделать рабочими, ну чтоб дату время вводить. А то сделал через атрибуты и не допру как стилизовать, потому что placeholder задан, но не отображается.
Р
Мне слайдер на мини и самому не очень понравился, да и многие нюансы, тот же ивент , просто не заморочился, а вот по якорю и контейнеру, я как то не подумал. А и в формах, хотел уточнить, выходит их только с помощью js можно сделать рабочими, ну чтоб дату время вводить. А то сделал через атрибуты и не допру как стилизовать, потому что placeholder задан, но не отображается.
Честно говоря, вообще впервые вижу необходимость вводить дату и время. Человек же не может заранее знать когда там свободно. Ты уверен, что это было в макете? Куда логичнее там смотрится email и телефон.
E
Да там значок календаря и часы
Р
Хм, можно нагуглить библиотеку, либо написать решение на жс, но это геморрой в данном случае. Думаю, библиотеки должны быть.
E
Про карты, подключить не проблема, а вот чтобы стилизовать нужен Гугл ключ, ты с помощью ключа подключаешь?
Р
Про карты, подключить не проблема, а вот чтобы стилизовать нужен Гугл ключ, ты с помощью ключа подключаешь?
Я вроде не писал про карту.
Р
Да, нужен апи-ключ, со мной им любезно поделилась коллега
E
Просто по макету карта чёрно-белая и обозначена специфическим значком.