fix(grid-pinning): remove hide call in scrollToRow - master

[IMinchev64]

Closes #16493

Additional information (check all that apply):

• Bug fix
• New functionality
• Documentation
• Demos
• CI/CD

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

[mddragnev]

@skrustev @IMinchev64 The change is fine. However, if the action strip is set as menu this behavior is not perfect IMO.
I think that when having the action strip as menu we should close the menu on every action click
https://github.com/user-attachments/assets/635939be-2a58-4f49-830e-0fa6898cc35a

[IMinchev64]

@mddragnev I added a condition to continue hiding the strip when the action strip is a menu.

Note that testing this in elements would be a bit of a hassle currently as:

1. The fix for this issue ([Elements] igc-action-strip stays hidden / not visible #16834) would need to be merged in both master and 21.0.x as the action strip is broken in elements for the respective branches
2. Even when the action strip works as a whole, using it as a menu is broken in elements as the moment we try to hover over the menu, the action strip gets hidden. This should be logged and fixed separately.

[mddragnev]

The issue still reproduces when the record you try to scroll to is out of view

[IMinchev64]

The issue still reproduces when the record you try to scroll to is out of view

I am failing to reproduce this scenario with the same result

[IMinchev64]

Replaced the unconditional action-strip hide/reset on vertical scroll with a conditional check: keep it only when the current context row is pinned, still connected to the DOM, and hovered; otherwise keep the previous hide + context reset behavior. This preserves virtualization cleanup while preventing the strip from disappearing during pinned-row jump/scroll scenarios.

[Copilot]

Pull request overview

Fixes a regression where the grid Action Strip closes when using the row-pinning “jump” action (scroll), especially for pinned rows.

Changes:

• Update grid vertical scroll handling to avoid hiding the Action Strip when scrolling a hovered pinned row.
• Update the pinning actions “scrollToRow” behavior to only auto-hide the Action Strip when the actions are rendered as menu items.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
projects/igniteui-angular/grids/grid/src/grid-base.directive.ts	Keeps the Action Strip visible during vertical scroll when the current context is a hovered pinned row.
projects/igniteui-angular/grids/core/src/grid-actions/grid-pinning-actions.component.ts	Prevents auto-hiding the Action Strip on “jump” in button mode; preserves menu behavior.

[dkamburov]

@IMinchev64 please click on resolve conversation button if the review comments are ready.