Дополняет статью про формы #5000
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
solarrust
reviewed
Nov 28, 2023
There was a problem hiding this comment.
Круто подмечено, спасибо!
Кстати, сперва засомневалась, что поведение формы ровно такое, как ты описываешь. Сделала демку и убедилась, что реально поля не очищаются при сабмите, а диалог закрывается. Приятно узнавать что-то новое =)
Пара моментов, которые меня смущают:
- В общей фразе, описывающей
methodговорится, что это «способ» отправки данных на сервер. Аdialogбудто бы не про это. Может быть попробуем подобрать более обтекаемую формулировку для общей описательной фразы? - Складывается ощущение, что данные из формы, расположенной внутри диалога, как бы теряются, раз запроса на сервер не происходит. Может допишем что-то вроде «данные формы можно отправить при помощи JavaScript»? Как вариант, можно сослаться на статью «Работа с формами».
Очень жду твоего мнения 🙏
Co-authored-by: Alena Batitskaia <[email protected]>
|
@solarrust внес правки |
solarrust
reviewed
Nov 30, 2023
solarrust
approved these changes
Nov 30, 2023
Превью контента из 4c96b9d опубликовано. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Описание
Closes #4981
Чек-лист
/css/color/,/tools/json/,/tools/gulp/#kak-ponyat)images/example.png,demos/example/,../demos/example/)