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 backspace handling on MacOS and allow null view parameter to NewFrame #2818

Closed
wants to merge 1 commit into from

Conversation

DiligentGraphics
Copy link
Contributor

This fixes the issue with backspace handling (#2817).

It also allows passing null view to ImGui_ImplOSX_NewFrame(). This adds flexibility as the method may be called from thread other than main as methods of NSView can only be accessed from the main thread.

Allow null view passing as parameter to ImGui_ImplOSX_NewFrame
@DiligentGraphics DiligentGraphics changed the title Fixed backspace handling on MacOS and allow null null view parameter to NewFrame Fixed backspace handling on MacOS and allow null view parameter to NewFrame Oct 1, 2019
@ocornut
Copy link
Owner

ocornut commented Oct 11, 2019

Hello @DiligentGraphics

Thanks for the PR!
It looks good, will merge.

I will change
if (!io.KeyCtrl && !((c >= 0xF700 && c <= 0xFFFF) || c == 127))
to
if (!io.KeyCtrl && !(c >= 0xF700 && c <= 0xFFFF) && c != 127)

ocornut added a commit that referenced this pull request Oct 11, 2019
@ocornut
Copy link
Owner

ocornut commented Oct 11, 2019

Merged now, thanks!

@ocornut ocornut closed this Oct 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants