Merge lp:~nikwen/ubuntu-calculator-app/reboot-white-splash-screen into lp:ubuntu-calculator-app

Proposed by Niklas Wenzel
Status: Merged
Approved by: Bartosz Kosiorek
Approved revision: 159
Merged at revision: 163
Proposed branch: lp:~nikwen/ubuntu-calculator-app/reboot-white-splash-screen
Merge into: lp:ubuntu-calculator-app
Diff against target: 9 lines (+1/-1)
1 file modified
app/ubuntu-calculator-app.desktop.in.in (+1/-1)
To merge this branch: bzr merge lp:~nikwen/ubuntu-calculator-app/reboot-white-splash-screen
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Riccardo Padovani Approve
Review via email: mp+255912@code.launchpad.net

Commit message

Add white splash screen

Description of the change

Add white splash screen.

Why I used a white splash screen:

1) Showing a fake MainView with a header does not make any sense since the application hides the header on startup.
2) Displaying another image: Why?
3) The application uses a white background so it makes sense to make the splash screen white as well. (I stole the color from the clock app. :p)

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Niklas Wenzel (nikwen) wrote :

The Jenkins bot seems to be running all tests twice for me and failing the second time. Same behaviour could be seen on my other branch.

Revision history for this message
Riccardo Padovani (rpadovani) wrote :

I already reported the Jenkins fail, it tries to run both for trunk and for reboot.

Anyway, shouldn't be better to take a screenshot of the app without any calc and use it as splash screen? So it seems it has only to load calcs :-)

Revision history for this message
Niklas Wenzel (nikwen) wrote :

Thank you for your answer, Riccardo.
I sent a mail to nskaggs yesterday as well. Let's see what he says.

However, I wouldn't vote for using a screenshot as a splash screen for various reasons:

1) The screenshot wouldn't be resized (that's not supported by the splash screen). Now if you take a screenshot on a Nexus 4, it would be too big on the Aquaris while a one from the Aquaris would be too small on a Nexus 4.
2) The user would see the UI but it would be unresponsive. Not the best first impression.
3) You'd have to update the screenshot whenever you tweak the UI.

Revision history for this message
Riccardo Padovani (rpadovani) wrote :

While I don't think 2 and 3 are blockers, the first one is a problem. Thanks for the explanation, I approve this branch, so thanks also for the contribution :-)

review: Approve
Revision history for this message
Niklas Wenzel (nikwen) wrote :

You're welcome, and thanks for approving. :)

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

FAILED: Autolanding.
Approved revid is not set in launchpad. This is most likely a launchpad issue and re-approve should fix it. There is also a chance (although a very small one) this is a permission problem of the ps-jenkins bot.
http://91.189.93.70:8080/job/ubuntu-calculator-app-reboot-autolanding/97/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2574
        deb: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2574/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calculator-app-reboot-utopic-amd64-autolanding/32
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calculator-app-reboot-vivid-amd64-autolanding/97

review: Needs Fixing (continuous-integration)
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Ok, so merging failed because the old tests were finished before the reboot ones.
Should we try it again (it worked for yesterday's MP) or wait until the issue is fixed?

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Thanks for trying to get this merged. Seems like the system doesn't want us to do that. I suggest waiting until that Jenkins issue is fixed.

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

FAILED: Autolanding.
Approved revid is not set in launchpad. This is most likely a launchpad issue and re-approve should fix it. There is also a chance (although a very small one) this is a permission problem of the ps-jenkins bot.
http://91.189.93.70:8080/job/ubuntu-calculator-app-reboot-autolanding/99/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2586
        deb: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2586/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calculator-app-reboot-utopic-amd64-autolanding/34
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calculator-app-reboot-vivid-amd64-autolanding/99

review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Thanks for finally getting this merged. :)

Btw, Nicholas just notified me that the issue has been resolved. :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/ubuntu-calculator-app.desktop.in.in'
2--- app/ubuntu-calculator-app.desktop.in.in 2015-01-20 00:55:24 +0000
3+++ app/ubuntu-calculator-app.desktop.in.in 2015-04-11 22:09:18 +0000
4@@ -8,4 +8,4 @@
5 Type=Application
6 X-Ubuntu-Touch=true
7 X-Ubuntu-Default-Department-ID=accessories
8-
9+X-Ubuntu-Splash-Color=#F5F5F5

Subscribers

People subscribed via source and target branches