Skip to content

fix: DH-21744: Fix Advanced Filters in Pivots#1308

Open
vbabich wants to merge 3 commits intodeephaven:mainfrom
vbabich:fix-pivot-advanced-filters
Open

fix: DH-21744: Fix Advanced Filters in Pivots#1308
vbabich wants to merge 3 commits intodeephaven:mainfrom
vbabich:fix-pivot-advanced-filters

Conversation

@vbabich
Copy link
Collaborator

@vbabich vbabich commented Feb 20, 2026

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes advanced filters in pivots by adding hover state management to the PivotFilterMouseHandler class. The changes implement mouse move tracking and mouse leave handling for pivot column source filter boxes.

Changes:

  • Added onMove method to track mouse movement and set hover state for column source filters
  • Added onLeave method to clear hover state when the mouse leaves the grid
  • Implements logic to distinguish between column header area and regular filter bar area

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vbabich vbabich changed the title fix: DH-...: Fix Advanced Filters in Pivots fix: DH-21744: Fix Advanced Filters in Pivots Feb 20, 2026
@vbabich vbabich marked this pull request as ready for review February 20, 2026 18:59
@vbabich vbabich requested review from a team and dgodinez-dh and removed request for a team February 20, 2026 18:59
dgodinez-dh
dgodinez-dh previously approved these changes Feb 20, 2026
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.

3 participants