Merge lp://qastaging/~mcintire-evan/ubuntu-terminal-app/splash-screen into lp://qastaging/~ubuntu-terminal-dev/ubuntu-terminal-app/reboot

PASSED: Continuous integration, rev:174
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1019/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/571/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1019/rebuild

Works perfectly for me

Sorry Evan for being that hateful guy, but the branch surely needs a fix. :)

Line 9 of the diff should be:
_X-Ubuntu-Splash-Title=Terminal

Note the underscore before "X-Ubuntu-Splash-Title". That way "Terminal" can be properly translated into user's language.

As for the rest, I'm not sure if we really want to show an header since we don't have any header in the main page of terminal-app.
This is anyway not a important question, so feel free to choose the design you prefer :)

I agree we should not have a header where the app has none.

Thank you, Evan, for going through the old bugs and trying to fix them!

Here are two screenshots comparing the old and the newer version:
https://drive.google.com/file/d/0B19TeQHX9rTYOVVIT0pPRmU2MW8/view?usp=sharing
https://drive.google.com/file/d/0B19TeQHX9rTYNVRzMW11Qzd6clE/view?usp=sharing

What I like about the old one is that it features the app icon and the application name neatly centered with the spinner below. I have to say that I really like the awesome terminal app icon on that dark background.

What I like about the newer version is the new background color. I think it looks more friendly than the old one. However, I have to agree with Stefano and Alan that an application header where the app uses none looks a bit out of place.

What do you think about disabling the header again but keeping the color? :)

That's what I think would look best, and additionally you wouldn't have to worry about the translations thing. ;)

Alright - sounds good to me :) I do like the version without the header better too, give me just a moment

PASSED: Continuous integration, rev:175
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1026/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/578/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1026/rebuild

Great, LGTM. Thank you!

http://people.canonical.com/~alan/screenshots/device-2016-02-11-215750.png

Looks good to me!

Thanks, Evan!