Feature #1090

Do not set splash screen as system modal

Added by Joel Palmius about 2 years ago. Updated over 1 year ago.

Status:Fix exists, needs testingStart date:01/06/2017
Priority:NormalDue date:
Assignee:Joel Palmius% Done:

90%

Category:MakeHuman GUI
Target version:MakeHuman 1.1.2

Description

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.

History

#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.

#2 Updated by Aranuvir # over 1 year ago

  • Status changed from Accepted to Fix exists, needs testing
  • % Done changed from 0 to 90

Could be controlled by a command line option (-s/--splashontop). New default will be "not always on top".

#3 Updated by Jonas Hauquier over 1 year ago

Too complex, not much use.
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.

#4 Updated by Joel Palmius over 1 year ago

I've always hated the nudity warning message box. I think it implies that the user is an idiot. I'm all for removing it.

#5 Updated by Aranuvir # over 1 year ago

Quote Jonas:

Too complex

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 ;))

#7 Updated by Aranuvir # over 1 year ago

We could implement another popup, that lists all command line options, to make sure we don't forget them...

#8 Updated by Jonas Hauquier over 1 year ago

Aranuvir (Moderator) wrote:

We could implement another popup, that lists all command line options, to make sure we don't forget them...

No, makehuman -h is used for that ;)

#9 Updated by Aranuvir # over 1 year ago

Sorry, forgot the <joke> </joke> tags on my last post.

#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