Skip to content

Conversation

@mzanev
Copy link
Contributor

@mzanev mzanev commented Nov 3, 2025

We should not merge this until Q4

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Native Angular Report Viewer Events documentation to include proper event arguments and type definitions. The changes add missing event handler implementations and introduce documentation for new interactive action and tooltip events.

Key Changes:

  • Updated existing event handlers to accept and properly type their event arguments
  • Added four new event sections: interactiveActionEnter, interactiveActionLeave, toolTipOpening, and toolTipClosing
  • Added necessary import statements for event argument types from @progress/telerik-common-report-viewer

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

mzanev and others added 10 commits November 3, 2025 17:54
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion/native-angular-report-viewer/api-reference/events.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

@dnikolov-prg dnikolov-prg left a comment

Choose a reason for hiding this comment

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

Thanks for updating the events article!

@dnikolov-prg dnikolov-prg merged commit 4f18cb3 into master Nov 4, 2025
@dnikolov-prg dnikolov-prg deleted the mzanev/update-narv-events branch November 4, 2025 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants