[jboss-jira] [JBoss JIRA] (DROOLS-4845) Fix error-popover position on simulation/model grids

[ https://issues.redhat.com/browse/DROOLS-4845?page=com.atlassian.jira.plug... ] Gabriele Cardosi edited comment on DROOLS-4845 at 12/5/19 10:49 AM: -------------------------------------------------------------------- [~yamer] This is the snippet {code:java} /* It determines if the popover should be draw on the RIGHT or in the LEFT of the cell */ if (xPosition + POPOVER_WIDTH > scenarioGrid.getLayer().getWidth()) { xPosition = (int) cellXYMiddleCoordinates.getX() - xMiddleWidth; position = PopoverView.Position.LEFT; } {code} Error could be related to 1) {code:java}scenarioGrid.getLayer().getWidth(){code} - maybe it is not the correct "container" to consider 2) not considering {code:java}int scrollX = scenarioGridPanel.getScrollPanel().getElement().getScrollLeft();{code} in the `if` evaluation 3) both/any was (Author: gabriolo): [~yamer] This is the snippet {code:java} /* It determines if the popover should be draw on the RIGHT or in the LEFT of the cell */ if (xPosition + POPOVER_WIDTH > scenarioGrid.getLayer().getWidth()) { xPosition = (int) cellXYMiddleCoordinates.getX() - xMiddleWidth; position = PopoverView.Position.LEFT; } {code} Error could be related to 1) `scenarioGrid.getLayer().getWidth()` - maybe it is not the correct "container" to consider 2) not considering ` int scrollX = scenarioGridPanel.getScrollPanel().getElement().getScrollLeft();` in the `if` evaluation 3) both/any -- This message was sent by Atlassian Jira (v7.13.8#713008)