Merge lp://staging/~seb128/gnome-control-center-unity/swap-color-use-gdkcolor into lp://staging/gnome-control-center-unity

Proposed by Sebastien Bacher
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: 31
Merged at revision: 31
Proposed branch: lp://staging/~seb128/gnome-control-center-unity/swap-color-use-gdkcolor
Merge into: lp://staging/gnome-control-center-unity
Diff against target: 55 lines (+14/-14)
1 file modified
appearance/cc-appearance-panel.c (+14/-14)
To merge this branch: bzr merge lp://staging/~seb128/gnome-control-center-unity/swap-color-use-gdkcolor
Reviewer Review Type Date Requested Status
Łukasz Zemczak (community) Approve
Michael Terry Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+189309@code.staging.launchpad.net

Commit message

Use GdkColor types, for color swapping, since that's what
libgnome-desktop uses. That's a fix similar to the one done previous to
the color selection code.

Description of the change

Use GdkColor types, for color swapping, since that's what
libgnome-desktop uses. That's a fix similar to the one done previous to
the color selection code.

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

That's similar to https://code.launchpad.net/~seb128/gnome-control-center-unity/use-correct-color/+merge/168800 which was fixed earlier in the cycle, the color swapping code had the same issue

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Terry (mterry) wrote :

Makes sense.

review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Works like a charm.

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

to all changes: