neos-ui icon indicating copy to clipboard operation
neos-ui copied to clipboard

ReactUi: Validator in node creation dialogue

Open ursulaklinger opened this issue 7 years ago • 3 comments

Description

We have a node creation dialogue a single line text and a select box. Both have a notEmptyValidation. It is possible to submit the dialogue by hitting return even though the validator disables the submit button. Furthermore, the Validator for the selectbox only works if you have touched the selectbox once: If you never clicked on it, you can submit the form even though the selectbox is empty. However, if you click on the selectbox or select an element and then remove it again, the validator will behave as expected and disable the submit button. It also fails to display the red popup that should alert the user that the validation has failed.

Steps to Reproduce

  1. Add a creation dialogue to your site with a single line text and a select box
  2. add NotEmptyValidators to those properties
  3. try and create a page without correct input

Expected behavior

See description

Actual behavior

See description

Affected Versions

Neos: 3.2

Flow: 4.2.5

You can get this information by running composer show or using the package management module within Neos.

ursulaklinger avatar Nov 15 '17 15:11 ursulaklinger

Hi @ursulaklinger. First of all thanks for reporting and sorry that it took so long to answer you.

Do you still have this issue with the latest release of the neos-ui ? If yes please report it here https://github.com/neos/neos-ui/issues

mgoldbeck avatar Mar 06 '18 13:03 mgoldbeck

Thank you for the bug report. Please post the issue in the React UI issues at https://github.com/neos/neos-ui/issues and close this issue.

Thanks a lot!

rolandschuetz avatar Apr 12 '18 19:04 rolandschuetz

I will move this to the Ui repo, but idk if this is still an issue - cannot remember.

mhsdesign avatar Jul 19 '22 20:07 mhsdesign

Just tested this on 7.3 and was not able to reproduce. So, I'll close.

grebaldi avatar May 02 '23 13:05 grebaldi