-
-
Notifications
You must be signed in to change notification settings - Fork 290
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
Added back ImGui MKB Input functions, and added new "Overlay Hotkey" category (Issue #957) #958
Added back ImGui MKB Input functions, and added new "Overlay Hotkey" category (Issue #957) #958
Conversation
Will need to have proper look tomorrow but I would want some changes definitely. Overall doesnt seem bad though and seems to cover #957 quite nicely 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just some quick review but i want a more proper look as said, this is really surface level
Do overlay keys not block ImGui input? Was this tested fully? I have a feel it may forbid from typing some letters but yeah, that is one of the things I need to properly "review"... And another thing - did you verify that input functions of ImGui are not accessible outside of onDraw right? Think we move whole ImGui table outside of onDraw but better be sure... Another thing I need to properly look at... Guess other things will get resolved when you change it according to comments but will write them down tomorrow if they wont change in the meantime. |
One more note following Discord messages - ImGui keyboard functions may not work with other layout than QWERTY I realised, would need to be checked... If it doesnt function with other layout, would mark it as limitation for now as Im unsure now if it works with other layouts without changes in imgui_impl/win32.cpp (which this shouldnt touch ideally... that should be left for some other PR if it is an issue...) |
|
…nd changes to Bindings.cpp to account for new structure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
registerOverlayHotkey(id, name, callback)