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

Fixed a bug that freezed when changing DocumentPane Orientation #224

Merged
merged 1 commit into from
Jan 13, 2021

Conversation

sukamoni
Copy link
Contributor

@sukamoni sukamoni commented Jan 4, 2021

When dropping a DoucumentPane onto another DoucumentPane, the process freezes under certain condition
Occurs in the following situations

  1. Launch TestApp
  2. Document1 drop to target Right
  3. Document1 drop to target Top
  4. Document2 drop to target Document1
  5. freeze

It seems to be an old problem, and that part has been fixed

@Dirkster99 Dirkster99 merged commit 50f9be2 into Dirkster99:master Jan 13, 2021
@Dirkster99
Copy link
Owner

Hi Sukamoni

I had to undo your PR because I found through testing that it seems to introduce new issues - I started the MLibTest Tool and used the below workflow and ended in a crash which did not appear without your PR :-(

Would you be able to review this crash and maybe do a bit more testing before trying to apply this fix again?

  • Start MLibTest and Click File>New to create 3 new documents
    Untitled

  • Drag the last Document into the Right drop target
    Untitled1

  • Result: Crash in DocumentPaneDropTarget class
    Untitled3

@sukamoni
Copy link
Contributor Author

Hi Dirkster99
fix again PR #229

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.

None yet

2 participants