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

IxSelect with form validation is broken #1573

Closed
2 tasks done
tobiask84 opened this issue Nov 24, 2024 · 1 comment
Closed
2 tasks done

IxSelect with form validation is broken #1573

tobiask84 opened this issue Nov 24, 2024 · 1 comment
Labels
type: duplicate This issue or pull request already exists

Comments

@tobiask84
Copy link

Prerequisites

  • I have read the Contributing Guidelines.
  • I have not leaked any internal/restricted information like screenshots, videos, code snippets, links etc.

What happened?

IxSelect with is-invalid is buggy. This has already been mentioned here:
#1389

The ticket was moved to closed but I cant find any changes in the docs of form validation? How is this used? It still looks like in the attached screenshot to me:
image

What type of frontend framework are you seeing the problem on?

React

Which version of iX do you use?

v.2.5.0

Code to produce this issue.

This is version 2.5 of ix. Here is the code in react:

        <IxSelect
          className="is-invalid"
          ...

I cannot see selects being mentioned here:
https://ix.siemens.io/docs/controls/validation
@tobiask84 tobiask84 added the triage We discuss this topic in our internal weekly label Nov 24, 2024
@danielleroux danielleroux added type: duplicate This issue or pull request already exists and removed triage We discuss this topic in our internal weekly labels Nov 25, 2024
@danielleroux
Copy link
Collaborator

The issue #1389 was closed because its merged on main already (PR: #1318). In the PR you will find another linked PR which contains the information about which version the library will take for the release:
image
e.g #1480
The issue will be part of the next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants