Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Add morestep fieldset validation #1090

Open
wants to merge 10 commits into
base: typo3-v12
Choose a base branch
from

Conversation

denislorch
Copy link
Contributor

We need the option to validate the current (visible) fieldset on multistep forms.
The validation is done clientsided in JavaScript on page switch and scrolls to the first visible error if exist.
To simplify the usage, the setting validation.morestep.fieldset was added.

As a nice to have improvement, a simple scroll into view for multistep forms on page switch was added, to avoid manual user scrolling if the height of the fieldsets differ.

* to validate those fields in multistepform for example
* if the data-powermail-morestep-validate attribute is set to "true" on morestep button (new option)
* we validate the visible fields and scroll to the first error in error case
* without it's problamtic if the fieldsets differ in height
* if true, the user has to scroll manually to input the new fieldset
* scroll on navigation click, not on btn-group link above
@Mikel2411
Copy link

The release of feature/visible-field-validation looks good. But unfortunately the form.min.js does not contain the Javascript adjustments.

@denislorch
Copy link
Contributor Author

The form.min.js relates to the changing master branch, the file should be generated if the PR is accepted as described here https://github.com/in2code-de/powermail/tree/master/Resources/Private/Build, before the merge. The Feature was implemented for the TYPO3 v12 early access pro branch here https://github.com/in2code-pro/powermail/pull/81.

@denislorch denislorch changed the base branch from master to typo3-v12 December 2, 2024 11:44
@denislorch
Copy link
Contributor Author

I updated the PR to the latest commit (typo3-v12 branch) and validated it again.
Let me know if an updated form.min.js is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants