Add HTML5 'required' attribute to form widgets. #28
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.
Hello @artyom-beilis,
In current version, CppCMS does support 'disabled' attribute on form widgets. But, if a developer would like to add 'required' attribute to form widgets in templates, he/she is required to write that part manually each time.
This commit provides code for 'required' attribute.
To not break backward compatibility, required flag is not checked if the widget have non_empty(). But this may cause confusion for new developers, so what's your opinion? Should CppCMS check even if non_empty() function is present?
Thanks.