Skip to content

Conversation

@labkey-tchad
Copy link
Member

Rationale

The editable grid's activateCellUsingDoubleClick and shiftSelectRange methods don't scroll correctly which can cause errors when using particularly long field names.

org.openqa.selenium.interactions.MoveTargetOutOfBoundsException: Move target (1289, 610) is out of bounds of viewport dimensions (1280, 891)
  [...]
  at app//org.labkey.test.WebDriverWrapper.doubleClick(WebDriverWrapper.java:2929)
  at app//org.labkey.test.components.ui.grids.EditableGrid.activateCellUsingDoubleClick(EditableGrid.java:660)
  at app//org.labkey.test.tests.component.EditableGridTest.testPasteIntoMultiLine(EditableGridTest.java:593)
java.lang.AssertionError: not all rows are selected
  [...]
  at org.labkey.test.util.DeferredErrorCollector.verifyTrue(DeferredErrorCollector.java:290) [uiTest/:?]
  at org.labkey.test.tests.component.EditableGridTest.testShiftClick(EditableGridTest.java:414) [uiTest/:?]

Related Pull Requests

  • N/A

Changes

  • Update EditableGrid and EditableGridTest to handle long field names
  • Clean up checkbox handing in EditableGrid

@labkey-tchad labkey-tchad merged commit ace1198 into release25.8-SNAPSHOT Aug 7, 2025
8 checks passed
@labkey-tchad labkey-tchad deleted the 25.8_fb_handleBigEditableGrid branch August 7, 2025 00:10
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