[igdmdimitrov]

Closes #17638

Approach: a single internal helper, preventImplicitSubmission (igniteui-angular/core), cancels Enter only on inputs that belong to a form and would submit it (not button, submit, reset, image, file or color inputs). It reads the focused element through event.composedPath(), so elements inside shadow DOM are handled correctly. It runs on keydown and keypress, in the capture phase on the grid host. keypress is needed because a date picker that selects a day on Enter moves focus back to its input before the browser submits.

Used by: IgxGridBaseDirective (so it covers the tree, hierarchical and pivot grids too; the spec only exercises IgxGrid), igx-column-actions, igx-pivot-data-selector and igx-query-builder. The query builder covers the advanced filtering dialog, including when it's standalone or opened with custom overlay settings. The snackbar and the query builder buttons now havetype="button".

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

How Has This Been Tested?

• Unit tests
• Manual testing
• Automated e2e tests

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]

Copilot review overview

🟢 Approval recommended

The focused implementation correctly resolves the reported behavior and includes appropriate regression coverage.

Review effort: Balanced
Findings: None

What changed in this PR

Prevents the snackbar action button from implicitly submitting a containing form.

Changes:

• Sets the default action button’s type to button.
• Adds unit coverage for the button type.
• Documents the fix in the changelog.

File	Description
snackbar.component.html	Adds the explicit button type.
snackbar.component.spec.ts	Verifies the rendered type attribute.
CHANGELOG.md	Records the form-submission fix.

---

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

[mddragnev]

@igdmdimitrov
The issue still reproduces. Add a toolbar with the pinning or hiding action, open it, type something in the search input and press Enter. The page refreshes.

With type="button" on the snackbar, the form no longer has a submit button. But the HTML spec still allows implicit submission when a form with no submit button has only one text-like input: pressing Enter in that input submits the form. The column actions search input is that one input, so Enter submits the form.

Adding type="button" to individual buttons won't cover this. We need a more general fix that cancels the default action of Enter for inputs inside the grid.

[viktorkombov]

Verified locally on 8153548: the new spec plus the snackbar, query builder and pivot selector specs pass (194/194), the grid filtering/editing/toolbar/keyboard-nav suites pass (697/697), and with the production changes reverted 8 of the 11 new tests fail, so the spec guards the fix. Also checked a standalone igx-grid-excel-style-filtering inside a form: its custom dialog is attached through the grid body view container, so the grid listener covers it too. Two non-blocking nits inline.

[Copilot]

🟡 Changes recommended

The new keyboard bindings use @HostListener, which repository guidelines prohibit, and should be moved to the component’s host metadata.

1 open issue

• Use host metadata instead of @​HostListener · New

_{Review effort: Balanced}

[viktorkombov]

Rechecked the current head. Please strengthen the focused regression coverage below.