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

beforeDragMoveEnd and beforeDragResizeEnd has not correct event ids #7059

Closed
chuckn0rris opened this issue Jun 22, 2023 · 5 comments
Closed
Assignees
Labels
bug Something isn't working forum Issues from forum high-priority Urgent to have fixed resolved Fixed but not yet released (available in the nightly builds)
Milestone

Comments

@chuckn0rris
Copy link

Forum post

Add to basic demo

    listeners: {
        beforeDragMoveEnd: async ({ eventRecord, validation }) => {
            console.log(eventRecord.id);
            return false;
        },
        beforeDragResizeEnd: async ({ eventRecord, validation }) => {
            console.log(eventRecord.id);
            return false;
        }
    }

Try to resize, try to drag-n-drop. See different ids logged in console, but no one is correct.

@chuckn0rris chuckn0rris added bug Something isn't working forum Issues from forum labels Jun 22, 2023
@matsbryntse matsbryntse added the high-priority Urgent to have fixed label Jun 22, 2023
@canonic-epicure canonic-epicure self-assigned this Jun 29, 2023
@canonic-epicure
Copy link

Will be best if someone who is more in the calendar context will check this issue. It seems the calendar indeed creates a new empty record when drag starts and copies source record data into it. Then this "proxy" record is used when event is fired. There's probably a logical explanation of this and probably a fix is quite simple.

@chuckn0rris
Copy link
Author

Another report

@marciogurka
Copy link

User added a note about that here

@ExtAnimal
Copy link

I believe this ticket is fixed by this: https://github.com/bryntum/bryntum-suite/pull/7549

Assigning @mazzafabio so that he can check and close this ticket if that is the case.

@mazzafabio mazzafabio added in progress ready for review Issue is fixed, the pull request is being reviewed labels Aug 14, 2023
@isglass isglass added 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 Aug 30, 2023
@isglass isglass added this to the 5.5.3 milestone Aug 30, 2023
@kevinschweikert
Copy link

The issue is still there in 5.6.2 as i described in #7636

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working forum Issues from forum high-priority Urgent to have fixed resolved Fixed but not yet released (available in the nightly builds)
Projects
None yet
Development

No branches or pull requests

8 participants