Skip to content

feat(grids): add validationTooltipTrigger to control error tooltip - #17889

Open
mddragnev wants to merge 3 commits into
masterfrom
mdragnev/feat-17345
Open

mddragnev wants to merge 3 commits into
masterfrom
mdragnev/feat-17345

Conversation

@mddragnev

@mddragnev mddragnev commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Closes #17345
The Grid Validation Specification has been updated with the new option.

Description

Adds a validationTooltipTrigger input to the grids (active by default, or hover) that controls when the validation error tooltip of an invalid cell is displayed.

Motivation / Context

Today the error tooltip opens automatically whenever an invalid cell becomes active, including while the user is still typing in it, so it can cover grid content. We are keeping the validation behavior but add an option to show the message only when hovering the error icon.

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:

How Has This Been Tested?

  • Unit tests
  • Manual testing
  • Automated e2e tests

Test Configuration:

  • Angular version:
  • Browser(s):
  • OS:

Screenshots / Recordings

Checklist:

  • All relevant labels 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

Add the validationTooltipTrigger input with values 'active' (default)
and 'hover'. With 'hover' the validation error tooltip of an invalid
cell is shown only while hovering the error icon, instead of also
opening when the cell becomes active or invalid while editing. The
invalid state, error icon and aria attributes are kept.

Isolate the validation test sample data per fixture so that committed
edits no longer leak between tests.

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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@mddragnev mddragnev added ❌ status: awaiting-test PRs awaiting manual verification grid: cell-editing grid: row-editing grid forms: validation Forms validation related, including ngModel.status aka VALID/INVALID/TOUCHED/PRISTINE etc. version: 22.3.x labels Oct 28, 2026
@mddragnev
mddragnev marked this pull request as ready for review October 28, 2026 13:29
@teodosiah teodosiah added ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification labels Oct 29, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

forms: validation Forms validation related, including ngModel.status aka VALID/INVALID/TOUCHED/PRISTINE etc. grid: cell-editing grid: row-editing grid version: 22.3.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Grid Validation: Control the tooltip visibility behavior

4 participants