← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

Re: [Merge] lp:~mcintire-evan/ubuntu-terminal-app/splash-screen into lp:ubuntu-terminal-app

 

Review: Needs Fixing

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. ;)
-- 
https://code.launchpad.net/~mcintire-evan/ubuntu-terminal-app/splash-screen/+merge/285673
Your team Ubuntu Terminal Developers is subscribed to branch lp:ubuntu-terminal-app.


References