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

imgui demo_ui: no more delay (no more mouse wiggling) #72318

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

Brambor
Copy link
Contributor

@Brambor Brambor commented Mar 11, 2024

Summary

None

Purpose of change

No more mouse wiggling in ImGui demo!
Fix #72215

Experimentally determined the timeout is in milliseconds, so I wrote in the docs.

Describe the solution

Set delay to 5. This allows for 200FPS (I get 70FPS) while not burning the PC like when I set it to 0.

Looking back, it was mentioned in the issue, but I didn't understand how to do it.

Describe alternatives you've considered

Set the delay to a different amount, like 16ms for 60 FPS. I feel like 50ms is too low, as it allows only 20FPS which might not be ideal for some animations (which are present in the ImGui demo).

Testing

Inputs & Focus update instantly and as expected.
Plots work without the mouse wiggling.

Additional context

I was working on #69831 and needed to fix the input delays. Since my starting point was ImGui demo, I came back to fix it.

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Mar 11, 2024
@Maleclypse Maleclypse merged commit ef537eb into CleverRaven:master Mar 11, 2024
23 of 28 checks passed
@Brambor Brambor deleted the imgui-no-delay branch March 11, 2024 21:51
SurFlurer added a commit to SurFlurer/Cataclysm-DDA that referenced this pull request Mar 19, 2024
…lay"

This reverts commit ef537eb, reversing
changes made to 22a7030.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Imgui Demo Delayed key presses
2 participants