Skip to content

Commit

Permalink
fix: Table overflow button has lower priority than grid tokens (deeph…
Browse files Browse the repository at this point in the history
…aven#1510)

* The mouse handler for the table "view full contents" button was called
after the mousehandler for cell tokens. This caused links inside the
cell to open if the cell overflow button overlapped with a link inside
the cell underneath it.
* Priority for GridTokenMouseHandler changed to be after the
CellOverflowMouseHandler.

* Fixes deephaven#1480
  • Loading branch information
georgecwan authored Sep 13, 2023
1 parent be6747c commit 32e6d20
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/grid/src/Grid.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ class Grid extends PureComponent<GridProps, GridState> {
new GridHorizontalScrollBarMouseHandler(600),
new GridScrollBarCornerMouseHandler(700),
new GridRowTreeMouseHandler(800),
new GridTokenMouseHandler(825),
new GridTokenMouseHandler(875),
new GridSelectionMouseHandler(900),
];

Expand Down

0 comments on commit 32e6d20

Please sign in to comment.