Merge lp://qastaging/~mar-kolya/compiz/fix-for-1188230 into lp://qastaging/compiz/0.9.10

Proposed by kolya
Status: Rejected
Rejected by: MC Return
Proposed branch: lp://qastaging/~mar-kolya/compiz/fix-for-1188230
Merge into: lp://qastaging/compiz/0.9.10
Diff against target: 12 lines (+1/-1)
1 file modified
plugins/decor/src/decor.cpp (+1/-1)
To merge this branch: bzr merge lp://qastaging/~mar-kolya/compiz/fix-for-1188230
Reviewer Review Type Date Requested Status
MC Return Disapprove
Review via email: mp+167782@code.qastaging.launchpad.net

Commit message

Fix copy/paste error introduced by rev 3728. Fixes LP #1188230

Description of the change

It looks like rev 3678 introduced a copy/paste style error - window's width is being changed using it's height. This causes windows to change their width when they are minimized/restored.

This trivial patch fixes this logical error.

To post a comment you must log in.
Revision history for this message
MC Return (mc-return) wrote :
Revision history for this message
kolya (mar-kolya) wrote :

Sorry about that - I looked for tickets describing similar bug, not for similar MPs :(.

Revision history for this message
MC Return (mc-return) wrote :

No need to apologize. ;)
I forgot to "Approve", but we have to wait for Sam for the final ok.

review: Approve
Revision history for this message
MC Return (mc-return) wrote :

Sam reverted r3728, so this is outdated now...

kolya, thanx a lot for your effort though -> fixes are always welcome :)

review: Disapprove

Unmerged revisions

3736. By kolya

Fix copy/paste error introduced by rev 3728. Fixes LP #1188230

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