[ scummvm-Bugs-1657322 ] GUI: 'Mass Add' button text does not revert after mass add

[SourceForge.net]

Bugs item #1657322, was opened at 2007-02-11 15:31
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418820&aid=1657322&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: GUI
Group: None
Status: Open
Resolution: None
Priority: 3
Private: No
Submitted By: Max Horn (fingolfin)
Assigned to: Eugene Sandulenko (sev)
Summary: GUI: 'Mass Add' button text does not revert after mass add

Initial Comment:
This was originally reported by joachim. This is for current SVN, on all platforms:

In the launcher, if you abort MassAdd, the button in the GUI is stuck with the "Mass Add.." text, even when Shift is not pressed anymore. The same happens if you hold shift, then open the "Options" dialog, then release shift, then close the options dialog again.


The problem of course is that we don't see the key up event in LauncherDialog::handleKeyUp, as the shift key is still pressed when we open the mass add confirm alert box (which will then see the "key up" event instead.

One simple way to fix this is to simply always revert to the "Add Game" label after the mass add code run. But that won't cover the other cases. Somewhat more generic would be to reset the text whenever we notice that we lost/gained focus. That still won't ensure that we stay in sync with the shift key status. To be able to do that, we need to keep track of the modifier state throughout multiple event loops. Hence, we likely need an OSystem::getModifierState method, or a similar thing in a hypothetical EventManager class (see my mail to scummvm-devel).


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418820&aid=1657322&group_id=37116