Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sorters should not always be removed on row drop #7901

Closed
bmblb opened this issue Nov 20, 2023 · 0 comments
Closed

Sorters should not always be removed on row drop #7901

bmblb opened this issue Nov 20, 2023 · 0 comments
Assignees
Labels
feature request large-account Reported by large customer OEM OEM customer resolved Fixed but not yet released (available in the nightly builds)
Milestone

Comments

@bmblb
Copy link

bmblb commented Nov 20, 2023

Reported by client.

Steps to reproduce on advanced demo:

  1. sort WBS column
  2. swap two neighbot leaf tasks around
  3. sorter is removed, but row stays in place

Sorter is removed because in general dropping the row would break sorting order. However, there are cases where that's not the case, which are:

  1. we sort by WBS, which is calculated based on the position we are moving task to
  2. we move task inside sorted subgroup (group of records with the same value on which sorter does not have affect)

Part of the problem is that cut/paste action does not remove the sorter, but essentially it is the same operation

@bmblb bmblb added feature request large-account Reported by large customer OEM OEM customer labels Nov 20, 2023
@isglass isglass self-assigned this Dec 4, 2023
@isglass isglass added in progress ready for review Issue is fixed, the pull request is being reviewed resolved Fixed but not yet released (available in the nightly builds) and removed in progress ready for review Issue is fixed, the pull request is being reviewed labels Dec 4, 2023
@isglass isglass added this to the 5.6.3 milestone Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request large-account Reported by large customer OEM OEM customer resolved Fixed but not yet released (available in the nightly builds)
Projects
None yet
Development

No branches or pull requests

2 participants