Skip to content

Reorder and Move Filters with Drag and Drop + Improve Multiple Files Drag and Drop - #2618

Merged
AmmarAbouZor merged 3 commits into
esrlabs:masterfrom
AmmarAbouZor:filters-dnd
Jul 20, 2026
Merged

AmmarAbouZor merged 3 commits into
esrlabs:masterfrom
AmmarAbouZor:filters-dnd

Conversation

@AmmarAbouZor

Copy link
Copy Markdown
Contributor

This PR closes: #2571

It adds support for drag and drop in filters side panel for running sessions providing the following:

  • Reordering for filters and charts via drag and drop
  • Move compatible items between filters and charts via drag and drop

This PR also includes:

  • Change drag and drop behavior in multiple files table view to match the behavior on filter adding highlighting on both top and bottom of items and following the relative position of mouse course when hovering over the item while dragging.
  • Check and remove stale edit state for filters when filters are removed from other UI components (like Library view)
  • Refactor filters.rs file splitting it into multiple sub-modules and removing low-signal unit tests

* Filters and charts can be reordered using drag and drop.
* Compatible items can be moved between filters and charts using drag
  and drop.
* Stale edit states in filters side panel UI is checked and removed.
* Split filters side panel view into multiple sub-modules.
* Check code style in the modules improving the code without logic
  changes.
* Remove low value unit tests
Align drop highlighting with pointer positions, allowing to draw the
highlight line on both top and bottom of rows to match drag and drop
behavior for filters in running sessions.

Painting the highlight will happen after finishing painting the table to
avoid table clipping.

Improve handling of faint highlight color for no-op destinations

Extend comments in file documenting the reasons for implementation
decisions.

@itsmesamster itsmesamster left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me. Kindly proceed.

@AmmarAbouZor
AmmarAbouZor merged commit 8797325 into esrlabs:master Jul 20, 2026
1 check passed
@AmmarAbouZor
AmmarAbouZor deleted the filters-dnd branch July 20, 2026 11:18
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.

Support for Drag and Drop for Filters

2 participants