Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix gray splash screen on Linux #100

Merged
merged 2 commits into from Dec 10, 2014
Merged

Conversation

martyngigg
Copy link
Member

Fixes trac issue #10741

Tester: Build and then start MantidPlot. You should now see the splash image for the whole duration of loading rather than a gray box and then the image popping up at the end of loading.

Ensures the image is painted promptly and does not wait for the event loop
to start. If the ApplicationWindow constructor takes too long then
the event loop paint events don't start and the image only gets painted
at the end.
Refs #10741
It only holds up startup even and doesn't seem to be required any
longer.
Refs #10741
@martyngigg martyngigg added this to the Release 3.3 milestone Dec 10, 2014
@eXeC64 eXeC64 self-assigned this Dec 10, 2014
eXeC64 added a commit that referenced this pull request Dec 10, 2014
@eXeC64 eXeC64 merged commit 56729dd into master Dec 10, 2014
@eXeC64 eXeC64 deleted the bugfix/10741_gray_splash_linux branch December 10, 2014 17:12
@rosswhitfield
Copy link
Member

This didn't fix it for me. I still get the grey splash screen on Ubuntu 14.10.

@martyngigg
Copy link
Member Author

@rosswhitfield - What Qt version is on 14.10? Could you add a screenshot? Thanks

I'll open a fresh issue as 14.10 isn't officially supported yet.

@rosswhitfield
Copy link
Member

Qt version 4.8.6. I believe Fedora 20 is also not fixed, if @peterfpeterson can confirm?

@martyngigg
Copy link
Member Author

Hmm, 14.04 has 4.8.5 so it is surprising it's different. I'll take a look on a VM.

@rosswhitfield
Copy link
Member

@sic reported it works with Fedora 20 and Qt 4.8.6. So maybe it's just a problem for my computer, if noone else has reported it.

@peterfpeterson
Copy link
Member

I can try again, but it didn't work on my f20 machine with Qt 4.8.6.

@peterfpeterson
Copy link
Member

Still doesn't work for me

@martyngigg
Copy link
Member Author

This is odd. Why would it work for @sic but not @peterfpeterson?

@eXeC64 eXeC64 removed their assignment Dec 11, 2014
@AndreiSavici
Copy link
Member

Works for me on Ubuntu 14.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants