Fix SetCursor(wxNullCursor) behaviour in wxOSX (PR #25814)
VZ
This should set the default cursor for the window instead of doing nothing.
Also simplify the code by overriding just the new WXUpdateCursor() member function instead of SetCursor() itself: this avoids the need for checking if the cursor has really changed.
Closes #25811.
You can view, comment on, or merge this pull request online at:
https://github.com/wxWidgets/wxWidgets/pull/25814
Commit Summary
- b72ba21 Fix SetCursor(wxNullCursor) behaviour in wxOSX
File Changes
(2 files)
- M include/wx/osx/window.h (3)
- M src/osx/window_osx.cpp (23)
Patch Links:
- https://github.com/wxWidgets/wxWidgets/pull/25814.patch
- https://github.com/wxWidgets/wxWidgets/pull/25814.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#25814)For the record, I'm still going to merge this even if it does not fix #25811, finally, see the discussion in #25813. But the current behaviour of SetCursor(wxCursor{}) still seems incorrect and this fixes it.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
Closed #25814.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#25814)Merged as the commit above.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.