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

p-table: A table with the LazyLoad=true and a filter for type "text" can leave its filter property value to "" instead of the expected null #14380

Closed
AIO1 opened this issue Dec 16, 2023 · 1 comment
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible

Comments

@AIO1
Copy link

AIO1 commented Dec 16, 2023

Describe the bug

It seems that there is a way to change the expected value for "text" filter type.
If at start, we load a table with lazy load, the columns which have a filter of type text will have the following in "value":
image

If we now try to filter by something we will see the value there:
image

If we now delete the "a" (we don't need to hit apply, by some reason it seems that it automatically applies if textbox is empty) from the dropdown we will see the following:
image

It was expected for it to be null again instead of "".

Environment

See below

Reproducer

No response

Angular version

17

PrimeNG version

17.1.0

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

v20.10.0

Browser(s)

No response

Steps to reproduce the behavior

No response

Expected behavior

No response

@AIO1 AIO1 added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Dec 16, 2023
@mertsincan
Copy link
Member

Hi,

So sorry for the delayed response! Improvements have been made to many components recently, both in terms of performance and enhancement. Therefore, this improvement may have been developed in another issue ticket without realizing it. You can check this in the documentation. If there is no improvement on this, can you reopen the issue so we can include it in our roadmap?
Please don't forget to add your feedback as a comment after reopening the issue. These will be taken into account by us and will contribute to the development of this feature. Thanks a lot for your understanding!

Best Regards,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible
Projects
None yet
Development

No branches or pull requests

2 participants