Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[[ Bug 21474 ]] Reset mouse stack after drag and drop#6750

Merged
livecodepanos merged 1 commit into
livecode:develop-9.0from
montegoulding:bugfix-21474
Oct 22, 2018
Merged

[[ Bug 21474 ]] Reset mouse stack after drag and drop#6750
livecodepanos merged 1 commit into
livecode:develop-9.0from
montegoulding:bugfix-21474

Conversation

@montegoulding

Copy link
Copy Markdown
Contributor

This patch resolves an issue where after a drag and drop operation the mouse stack
was being cleared in wmdragdrop. This resulted in a seemingly unresponsive stack
as mouse events are guarded by MCmousestackptr. As wmdragdrop is only called when
a window recieves a drag drop event we can ensure the mouse stack is correct by finding
the stack for the window and setting it to that.

This patch resolves an issue where after a drag and drop operation the mouse stack
was being cleared in `wmdragdrop`. This resulted in a seemingly unresponsive stack
as mouse events are guarded by `MCmousestackptr`. As `wmdragdrop` is only called when
a window recieves a drag drop event we can ensure the mouse stack is correct by finding
the stack for the window and setting it to that.
@montegoulding montegoulding added this to the 9.0.2-rc-1 milestone Oct 20, 2018
@livecodepanos

Copy link
Copy Markdown
Contributor

Tested locally.

@livecode-vulcan review ok 4834f8e

@livecode-vulcan

Copy link
Copy Markdown
Contributor

💙 review by @livecodepanos ok 4834f8e

livecode-vulcan added a commit that referenced this pull request Oct 22, 2018
[[ Bug 21474 ]] Reset mouse stack after drag and drop

This patch resolves an issue where after a drag and drop operation the mouse stack
was being cleared in `wmdragdrop`. This resulted in a seemingly unresponsive stack
as mouse events are guarded by `MCmousestackptr`. As `wmdragdrop` is only called when
a window recieves a drag drop event we can ensure the mouse stack is correct by finding
the stack for the window and setting it to that.
@livecode-vulcan

Copy link
Copy Markdown
Contributor

😎 test success 4834f8e

  • try-community-armv6-android-sdk26_ndk16r15: success
  • try-community-armv7-android-sdk26_ndk16r15: success
  • try-community-arm64-android-sdk26_ndk16r15: success
  • try-community-x86-android-sdk26_ndk16r15: success
  • try-community-x86_64-android-sdk26_ndk16r15: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos12.0: success
  • try-community-universal-ios-iphonesimulator12.0: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: success

@livecodepanos livecodepanos merged commit 75a368f into livecode:develop-9.0 Oct 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants