Ubuntu UI Toolkit

Merge lp://staging/~mterry/ubuntu-ui-toolkit/blue-spinner into lp://staging/ubuntu-ui-toolkit/staging

  1. blue-spinner
  2. Merge into staging
Proposed by Michael Terry
Status: Merged
Approved by: Zsombor Egri
Approved revision: 1324
Merged at revision: 2090
Proposed branch: lp://staging/~mterry/ubuntu-ui-toolkit/blue-spinner
Merge into: lp://staging/ubuntu-ui-toolkit/staging
Diff against target: 4 lines (+0/-0)
0 files modified
To merge this branch: bzr merge lp://staging/~mterry/ubuntu-ui-toolkit/blue-spinner
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Ubuntu SDK team Pending
PS Jenkins bot continuous-integration Pending
Review via email: mp+305001@code.staging.launchpad.net

This proposal supersedes a proposal from 2016-05-23.

Commit message

Make ActivityIndicator spinner blue instead of orange.

Description of the change

In my continuing quest to make Ubuntu color-consistent, I've gone ahead and updated the spinner asset from the old activity color (orange -- actually, we were using the old orange DD4814 instead of new orange E95420; tsk, tsk) to the new activity color (blue).

I just did this using Gimp's color exchange dialog. So no room for human error I hope. Looks good to me anyway.

To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote : Posted in a previous version of this proposal

When talking with Paty about this, she said: "The activity indicator/spinner should also be blue, but I've just checked with Rae (<email address hidden> // irc: raeshambrook) who is the visual designer in charge of the colour palette and she said there were some issues with that spinner. So she is going to check this and confirm which is the right asset. Sooo... I'll come back to you as soon as she does."

So I'm waiting on that.

Revision history for this message
Cris Dywan (kalikiana) wrote : Posted in a previous version of this proposal

Any update?

Revision history for this message
Michael Terry (mterry) wrote : Posted in a previous version of this proposal

From the linked bug:

Rae Shambrook (raecontreras) wrote on 2016-06-16:
A new blue spinner asset has been given to Zsombor to implement

So I guess this is out of my hands. Maybe this MP should be marked rejected and Zsombor should be poked.

Though if I had known how long that would take, I would have argued for just landing this palette swap MP in the meantime...

Revision history for this message
Zsombor Egri (zsombi) wrote : Posted in a previous version of this proposal

> From the linked bug:
>
> Rae Shambrook (raecontreras) wrote on 2016-06-16:
> A new blue spinner asset has been given to Zsombor to implement
>
> So I guess this is out of my hands. Maybe this MP should be marked rejected
> and Zsombor should be poked.
>
> Though if I had known how long that would take, I would have argued for just
> landing this palette swap MP in the meantime...

These assets look exactly the same as the ones you gave me, so I'd say let's get it in

Revision history for this message
Michael Terry (mterry) wrote : Posted in a previous version of this proposal

Ah, nm. Let's land this one.

<zsombi> mterry: well, my asset is nothing else than the previous one having the blue color, so in that sense if yours si the same, we can proceed with that :)
<mterry> Michael Terry zsombi: seriously? GUH! I had palette-swapped the existing asset in gimp like months ago and then Design said, "wait, we want to change the asset too"
<zsombi> mterry: I am patching the progress bar visuals right now, and had no plans to have the new activity indicator design, only the color - eventually
<mterry> Michael Terry zsombi: so yours looks like http://bazaar.launchpad.net/~mterry/ubuntu-ui-toolkit/blue-spinner/download/head:/spinner30.png-20130227062500-zxb8nbsbac23o3la-1/spinner%4030.png ?
<zsombi> mterry: they wanted us to rewrite teh whole component, with some fancy animation here'n'there, but just told them it's not gonna happen, there are way many more compponents that need love but the ActivityIndicator... so I got a new asset :D
<zsombi> mterry: haha, it does, let's get yours in then :)

Revision history for this message
Michael Terry (mterry) wrote :

Retargetted & merged staging.

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

PASSED: Continuous integration, rev:1324
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1143/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/5900/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1143/rebuild

review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

PASSED: Continuous integration, rev:1324
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/1352/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/5910/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/1352/rebuild

review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

PASSED: Continuous integration, rev:1324
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/1109/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/5911/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/1109/rebuild

review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

PASSED: Continuous integration, rev:1324
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1359/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/5914/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1359/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches