content icon indicating copy to clipboard operation
content copied to clipboard

Проблемы с демкой в html/input

Open jag-k opened this issue 3 years ago • 9 comments

При нажатии на кнопку отправки происходит POST запрос на html/input/demos/inputs/input1.php с данными browser=ie&comment=<данные из textarea> и выдаёт ошибку 404.

При фокусе на радио-кнопки не видно самого фокуса.

jag-k avatar Nov 15 '21 13:11 jag-k

Метко подмечено, спасибо большое! Возьмём в работу и поправим 🙌

@skorobaeus возьмёшь в работу?

solarrust avatar Nov 15 '21 14:11 solarrust

@solarrust если что, могу и я поправить))

jag-k avatar Nov 15 '21 15:11 jag-k

Давай! Так точно будет быстрее =)

solarrust avatar Nov 15 '21 15:11 solarrust

Тогда небольшой вопрос по отправке. Можно будет просто сделать отдельную страницу с выводом результатов запроса или pop-up?

jag-k avatar Nov 15 '21 15:11 jag-k

Попап — не очень хороший паттерн. Всех раздражают всплывающие окна =) А тут ещё и в айфрейме. Суть демки в самой форме, данные, отправляемые на сервер, не так важны. Можно скрыть форму и показать текст. Что-то вроде «форма отправлена 👍».

solarrust avatar Nov 15 '21 15:11 solarrust

Попап — не очень хороший паттерн. Всех раздражают всплывающие окна =) А тут ещё и в айфрейме. Суть демки в самой форме, данные, отправляемые на сервер, не так важны. Можно скрыть форму и показать текст. Что-то вроде «форма отправлена 👍».

Как вариант можно под формой добавить что отправилось

jag-k avatar Nov 15 '21 17:11 jag-k

Сделал предварительную версию. Так же накинул немного адаптива, но думаю, стоит ещё поработать

html/input demo with form result on desktop html/input demo with form result on mobile html/input demo on mobile
Демо после отправки формы Демо после отправки формы на мобильном устройстве Демо на мобильном устройстве

Что скажете, @solarrust?)

jag-k avatar Nov 15 '21 19:11 jag-k

Добавил вариант в виде отдельной страницы. Выглядит примерно так

Result of demo with filled form Result of demo with empty form
Отправлена заполненная форма Отправлена пустая форма

Работает при помощи GET запроса с формы на соседнюю "демку". Думаю оставить этот вариант

jag-k avatar Nov 15 '21 20:11 jag-k

На твой вопрос из твита отвечу, что решение о добавлении в контрибьюторы статьи принимаемся индивидуально. По демке отвечу в ПРе 🙂

solarrust avatar Nov 16 '21 05:11 solarrust