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

IgxGrid filtering event fires multiple times for column of type 'date' #10233

Closed
ddaribo opened this issue Oct 4, 2021 · 0 comments · Fixed by #10268
Closed

IgxGrid filtering event fires multiple times for column of type 'date' #10233

ddaribo opened this issue Oct 4, 2021 · 0 comments · Fixed by #10268
Assignees
Labels
🐛 bug Any issue that describes a bug filtering ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@ddaribo
Copy link
Contributor

ddaribo commented Oct 4, 2021

Description

The filtering (and correspondingly filteringDone) event fires multiple times for a date column.

  • igniteui-angular version: 11.1.26, 12.1.x
  • browser: any

Steps to reproduce

  1. Open the following sample
  2. Open the browser console
  3. Click the quick filter button of the "Order Date" column
  4. Select any date from the calendar
  5. Click anywhere to lose focus of the filled-in date input / Click the "Close" button

Result

The filtering event is fired on the following occasions:

  1. Once the quick filter button of the "Order Date" column is clicked when no filter is applied
  2. Twice when selecting a date from the calendar
  3. Once when losing focus of the filled-in date input

Expected result

The filtering event to be fired as for the other columns, f.ex. of type 'string'. To be fired only once when selecting a date from the calendar.

@ddaribo ddaribo added 🐛 bug Any issue that describes a bug filtering labels Oct 4, 2021
@zhosti zhosti self-assigned this Oct 6, 2021
@zhosti zhosti added 🛠️ status: in-development Issues and PRs with active development on them and removed 🆕 status: new labels Oct 6, 2021
@zhosti zhosti added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. and removed 🛠️ status: in-development Issues and PRs with active development on them labels Nov 8, 2021
DiyanDimitrov added a commit that referenced this issue Nov 15, 2021
fix(filtering): handle multiple events firing for column of type date #10233 - 12.2.x
kdinev added a commit that referenced this issue Nov 15, 2021
fix(filtering): handle multiple events firing for column of type date #10233
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Any issue that describes a bug filtering ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.
Projects
None yet
3 participants