Indicator Display Objects

Merge lp://staging/~larsu/ido/update-icon into lp://staging/ido/14.04

  1. update-icon
  2. Merge into trunk.14.04
Proposed by Lars Karlitski
Status: Merged
Approved by: Charles Kerr
Approved revision: 182
Merged at revision: 181
Proposed branch: lp://staging/~larsu/ido/update-icon
Merge into: lp://staging/ido/14.04
Diff against target: 113 lines (+52/-31)
1 file modified
src/idobasicmenuitem.c (+52/-31)
To merge this branch: bzr merge lp://staging/~larsu/ido/update-icon
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+214313@code.staging.launchpad.net

Commit message

idobasicmenuitem: update icon when the theme changes

Description of the change

idobasicmenuitem: update icon when the theme changes

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:181
http://jenkins.qa.ubuntu.com/job/ido-ci/63/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/ido-trusty-amd64-ci/21/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ido-trusty-armhf-ci/21/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ido-ci/63/rebuild

review: Needs Fixing (continuous-integration)
lp://staging/~larsu/ido/update-icon updated
182. By Lars Karlitski

idobasicmenuitem: don't export update_image()

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:182
http://jenkins.qa.ubuntu.com/job/ido-ci/64/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ido-trusty-amd64-ci/22
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ido-trusty-armhf-ci/22
        deb: http://jenkins.qa.ubuntu.com/job/ido-trusty-armhf-ci/22/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ido-ci/64/rebuild

review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

Looks good.

Omitting top approval only to wait for Jenkins

review: Approve

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