Do not set splash screen as system modal
|Status:||Fix exists, needs testing||Start date:||01/06/2017|
|Assignee:||Joel Palmius||% Done:|
|Target version:||MakeHuman 1.1.2|
The splash screen is always-on-top, system-wise, at least on windows 10.
This blocks other work, which is annoying when loading MH on a slow computer.
#1 Updated by Jonas Hauquier about 2 years ago
This is funny, because setting it to modal was actually done in response to complaints from Windows users.
There was some discussion about it causing problems if it was not modal. That might have had something to do with the dialog that popped up right after starting, though. Luckily that dialog has been removed since.
It's possible this discussion was on the old bugtracker on googlecode.
#5 Updated by Aranuvir # over 1 year ago
In which way? Implementation: No! Usability: Probably. Asking users to set a command line option could be ultimate fun...
It can move back to the old way, as long as we stay far away from any message popups on startup. They were annoying anyway.
This would be another solution. If someone should miss them, we could implement a command line option --show-silly-popups :D
#6 Updated by Jonas Hauquier over 1 year ago
Aranuvir (Moderator) wrote:
In which way?
Try and document that feature ;)
Even better: try documenting what the use is of such a feature, as concisely as possible ;)
This is the type of option that in a years' time you won't even remember yourself what it does (or what the hell you were thinking at the time you included it ;))
#10 Updated by Joel Palmius over 1 year ago
I've removed the nudity warning in https://github.com/makehumancommunity/makehuman/commit/ccd2a1a717619b91b7773398b0c5894a983a6624, but we should still decide upon the modality of the splash screen.
#11 Updated by Joel Palmius over 1 year ago
Maybe we could make the splash screen window modal rather than application modal, see http://doc.qt.io/qt-4.8/qdialog.html#modal-dialogs. Although it's not immediately apparent exactly which window it should be modal for.
Also available in: Atom