0ct0pu5/wesnoth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix addons connection dialog, and perhaps others

Browse source 
I am not entirely sure the workaround this is necessary for still
applies (setting the mouse button state after the first event pump),
but i'm also not sure how to test that it doesn't.
This commit is contained in:
Tommy 2022-07-31 00:31:39 +12:00
parent 461fd5fff5
commit 00ed0535a8
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/gui/widgets/window.cpp
View file

@ -510,6 +510,9 @@ int window::show(const unsigned auto_close_timeout)
// Start our loop drawing will happen here as well.
bool mouse_button_state_initialized = false;
for(status_ = status::SHOWING; status_ != status::CLOSED;) {
// Process and handle all pending events.
events::pump();
if(!mouse_button_state_initialized) {
/*
* The state must be initialize when showing the dialog.
@ -525,12 +528,13 @@ int window::show(const unsigned auto_close_timeout)
mouse_button_state_initialized = true;
}
// See if we should close.
if(status_ == status::REQUEST_CLOSE) {
status_ = exit_hook_(*this) ? status::CLOSED : status::SHOWING;
}
// This will rate limit to vsync
events::pump_and_draw();
// Update the display. This will rate limit to vsync.
events::draw();
}
}
catch(...)