Skip to content

RPM-6373: Unwanted scroll after Copy/Export action in frozen columns#481

Merged
OS-giulianasilva merged 1 commit intodevfrom
RPM-6373
Dec 17, 2025
Merged

RPM-6373: Unwanted scroll after Copy/Export action in frozen columns#481
OS-giulianasilva merged 1 commit intodevfrom
RPM-6373

Conversation

@OS-giulianasilva
Copy link
Contributor

@OS-giulianasilva OS-giulianasilva commented Dec 17, 2025

This pull request makes a minor improvement to the grid context menu feature. The change ensures that when focus is restored to a grid cell, the page does not scroll unexpectedly.

  • Updated the cell.focus() call in ContextMenu.ts to use { preventScroll: true }, preventing unwanted scrolling when restoring focus to the active grid cell.

Test Steps

Prepare the sample:

  1. Add a Grid with a large dataset
  2. Set the first column as frozen
  3. Set the cloneFrozenCells of the provider’s Grid to true: OutSystems.GridAPI.GridManager.GetActiveGrid().provider.cloneFrozenCells = true;
  4. Scroll to the grid to the bottom
  5. Right click on the last cell of the frozen column
  6. Click Export > to CSV or just Copy
  7. The Grid does not scrolls to the top.

Checklist

  • tested locally
  • documented the code
  • clean all warnings and errors of eslint
  • requires changes in OutSystems (if so, provide a module with changes)
  • requires new sample page in OutSystems (if so, provide a module with changes)

@OS-giulianasilva OS-giulianasilva requested a review from a team as a code owner December 17, 2025 14:16
@sonarqubecloud
Copy link

Copy link
Collaborator

@gnbm gnbm left a comment

Choose a reason for hiding this comment

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

LGTM

@OS-giulianasilva OS-giulianasilva merged commit b469491 into dev Dec 17, 2025
10 checks passed
@OS-giulianasilva OS-giulianasilva deleted the RPM-6373 branch December 17, 2025 15:27
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.

2 participants