Skip to content

fix(grid): preselect custom conditions in Excel-style filtering (22.2.x) - #17788

Merged
kdinev merged 3 commits into
22.2.xfrom
vkombov/fix-17636-22.2.x
Oct 23, 2026
Merged

kdinev merged 3 commits into
22.2.xfrom
vkombov/fix-17636-22.2.x

Conversation

@viktorkombov

Copy link
Copy Markdown
Contributor

Closes #17636

Description

Preselect custom conditions in the Excel-style filter dialog while preserving the built-in fallback.

Motivation / Context

Backport of #17786 to 22.2.x.

Type of Change (check all that apply):

  • Bug fix
  • New functionality
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation
  • Demos
  • CI/CD
  • Tests
  • Changelog
  • Skills/Agents

Component(s) / Area(s) Affected:

Grid, Tree Grid and Hierarchical Grid — Excel-style filtering.

How Has This Been Tested?

144 ESF tests passed, including three regressions. npm run lint:lib passed.

  • Unit tests
  • Manual testing
  • Automated e2e tests

Test Configuration:

  • Angular version: 22.2.0
  • Browser(s): Chrome Headless 151
  • OS: Windows 11

Screenshots / Recordings

N/A

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them
  • Accessibility (ARIA, keyboard navigation, focus management) has been verified

Copilot AI balanced review requested due to automatic review settings October 16, 2026 10:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The localized fix preserves built-in fallback behavior, includes targeted regression coverage, and has no identified blocking issues.

0 open findings

What changed in this PR

Backports #17786 to 22.2.x, fixing custom-condition preselection in Excel-style filtering for Grid, Tree Grid, and Hierarchical Grid.

Changes:

  • Resolves conditions from column filters before falling back to built-in operands.
  • Adds regressions for custom string and number conditions and built-in fallback.
  • Documents the fix in the changelog.
File Description
projects/​igniteui-angular/​grids/​grid/​src/​grid-filtering-ui.spec.ts Adds three regression tests.
projects/​igniteui-angular/​grids/​core/​src/​filtering/​excel-style/​excel-style-custom-dialog.component.ts Prioritizes column-defined filtering conditions.
CHANGELOG.md Records the filtering fix.

🧠 Review effort: Balanced


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@teodosiah teodosiah added 💥 status: in-test PRs currently being tested ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification 💥 status: in-test PRs currently being tested labels Oct 22, 2026
@kdinev
kdinev merged commit 8f12101 into 22.2.x Oct 23, 2026
5 checks passed
@kdinev
kdinev deleted the vkombov/fix-17636-22.2.x branch October 23, 2026 09:00
viktorkombov added a commit that referenced this pull request Oct 23, 2026
The fix is backported to 22.2.x in #17788, where it is listed under
22.2.4, so list it in the same section on master.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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 grid: excel-style-filtering grid version: 22.2.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants