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.

Sign up for GitHub

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

Jump to bottom

Form field generator shows irrelevant fields to configure #655

Closed
Toflar opened this issue Aug 21, 2019 · 1 comment
Closed

Form field generator shows irrelevant fields to configure #655

Toflar opened this issue Aug 21, 2019 · 1 comment
Assignees
@leofeyer
Labels
bug
Milestone
4.8

Comments

@Toflar
Copy link
Member

Toflar commented Aug 21, 2019

Affected version(s)

4.8

Description

The new min and max value in the form field settings are just fine. However, if I choose numeric values in the validation section, should not see the other two (maxlength and minlength) because they will be ignored anyway.

image

So I guess the rgxp would need to become a selector as well and we need some more palettes like text_digit etc. to show the correct fields.
@leofeyer leofeyer added the bug label Aug 21, 2019
@leofeyer leofeyer added this to the 4.8 milestone Aug 21, 2019
@leofeyer leofeyer self-assigned this Aug 21, 2019
leofeyer added a commit that referenced this issue Aug 21, 2019
@leofeyer
Hide the minlength/maxlength fields for numeric fields (see #655)
a90a86d
@leofeyer
Copy link
Member

leofeyer commented Aug 21, 2019 •
edited
Loading

Fixed in bb3c29a.

leofeyer added a commit that referenced this issue Aug 21, 2019
@leofeyer
Hide the minlength/maxlength fields for numeric fields (see #655)
bb3c29a
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@leofeyer leofeyer

Labels
bug
Projects
None yet
Milestone
4.8
Development

No branches or pull requests
2 participants
@Toflar @leofeyer