Skip to content

Examples: Dont re-enable a disabled cursor.#1202

Closed
PhilCK wants to merge 1 commit into
ocornut:masterfrom
PhilCK:glfw3_cursor
Closed

Examples: Dont re-enable a disabled cursor.#1202
PhilCK wants to merge 1 commit into
ocornut:masterfrom
PhilCK:glfw3_cursor

Conversation

@PhilCK
Copy link
Copy Markdown

@PhilCK PhilCK commented Jun 26, 2017

Might not be in the scope for the examples.

However if you capture the mouse imgui_impl_glfw.cpp will release it. Just added a check to stop that, brings it into line with the SDL example which doesn't suffer from this.

@ocornut ocornut changed the title Dont re-enable a disabled cursor. Examples: Dont re-enable a disabled cursor. Aug 11, 2017
ocornut added a commit that referenced this pull request Mar 20, 2018
…_HasMouseCursors, ImGuiBackendFlags_HasSetMousePos. (#787, #1495, #1202)
ocornut added a commit that referenced this pull request Mar 20, 2018
ocornut added a commit that referenced this pull request Mar 20, 2018
@ocornut
Copy link
Copy Markdown
Owner

ocornut commented Mar 20, 2018

Hello @PhilCK,

Sorry for my very late answer on this topic. In the past few months I've been touching related areas (mouse position / mouse cursors) as part of the work on viewport so this came back to my mind.

I merged your patch now, thanks for it. (I couldn't merge the exact branch/commit with author info as somehow git was causing me trouble as the repository got removed).

Recent changes also add a io.ConfigFlags field and the ImGuiConfigFlags_NoSetMouseCursor flag will disable any access to mouse cursor information in the back-end, so that's also a way for some applications to operate.
75c3793

Thanks again!
-Omar

@ocornut ocornut closed this Mar 20, 2018
ocornut added a commit that referenced this pull request Apr 6, 2018
…CursorChange. Followup to 75c3793 two weeks ago. (#787, #1495, #1202) + comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants