{"shas":[["4032e1ea4f88c4c8270743a2b1ba8d6b6a46b639","da0b52b0ab43721cda3399320ca940a5a0e571ee@bashasoliman.com","BASE: fixup - change display_wip_engine_warning config to be bool instead of string\n\nThanks fingolfin","Tarek Soliman"],["0f5d55435720fc87a4f966111cacd6f5458e773f","da0b52b0ab43721cda3399320ca940a5a0e571ee@bashasoliman.com","ENGINES: change warnUserAbout to be less terse and give users a chance to react to it\n\nOriginal comment from fingolfin:\n\nThe dialog shown to the user needs to be improved: First of, the warning is too terse. Secondly, there should be a way for the user to react to it.\n\nRegarding the first point, here's an alternative somewhat better text: \"WARNING: The game you are about to start is not yet full supported by ScummVM. As such, it is likely to run unstable, and any savestates you make could become invalid in a future ScummVM release.\"\n\nRegarding the second point: We could show two buttons, \"Start anyway\" and \"Cancel\".\n\nAdditionally, a checkbox \"Do not show this warning again\" might be nice -- but for that, GUI::MessageDialog is not powerful enough anymore, so we'd need a custom dialog, which is beyond the scope of this patch, I'd say.","Tarek Soliman"]],"repo":"tsoliman/scummvm","actor":"tsoliman","ref":"refs/heads/configure-enhancement-warning-clean","size":2,"head":"0f5d55435720fc87a4f966111cacd6f5458e773f","actor_gravatar":"112d759275e7cf194f5574988b4bdaae","push_id":30997758}